Commit 971505c
tls: do not refer to secureOptions as flags
Its confusing to have multiple names for the same thing, use
secureOptions consistently.
PR-URL: #9800
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>1 parent 761ae95 commit 971505c
1 file changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -12,9 +12,9 @@ var crypto = null; | |||
| 12 | 12 | const binding = process.binding('crypto'); | |
| 13 | 13 | const NativeSecureContext = binding.SecureContext; | |
| 14 | 14 | ||
| 15 | - function SecureContext(secureProtocol, flags, context) { | ||
| 15 | + function SecureContext(secureProtocol, secureOptions, context) { | ||
| 16 | 16 | if (!(this instanceof SecureContext)) { | |
| 17 | - return new SecureContext(secureProtocol, flags, context); | ||
| 17 | + return new SecureContext(secureProtocol, secureOptions, context); | ||
| 18 | 18 | } | |
| 19 | 19 | ||
| 20 | 20 | if (context) { | |
@@ -29,7 +29,7 @@ function SecureContext(secureProtocol, flags, context) { | |||
| 29 | 29 | } | |
| 30 | 30 | } | |
| 31 | 31 | ||
| 32 | - if (flags) this.context.setOptions(flags); | ||
| 32 | + if (secureOptions) this.context.setOptions(secureOptions); | ||
| 33 | 33 | } | |
| 34 | 34 | ||
| 35 | 35 | exports.SecureContext = SecureContext; | |
0 commit comments