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 <[email protected]> Reviewed-By: Michael Dawson <[email protected]>
nodejs · Dec 12, 2016 · a28e949 · a28e949
1 parent 475b8db
commit a28e949
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/lib/_tls_common.js b/lib/_tls_common.js
@@ -12,9 +12,9 @@ var crypto = null;
 const binding = process.binding('crypto');
 const NativeSecureContext = binding.SecureContext;
 
-function SecureContext(secureProtocol, flags, context) {
+function SecureContext(secureProtocol, secureOptions, context) {
   if (!(this instanceof SecureContext)) {
-    return new SecureContext(secureProtocol, flags, context);
+    return new SecureContext(secureProtocol, secureOptions, context);
   }
 
   if (context) {
@@ -29,7 +29,7 @@ function SecureContext(secureProtocol, flags, context) {
     }
   }
 
-  if (flags) this.context.setOptions(flags);
+  if (secureOptions) this.context.setOptions(secureOptions);
 }
 
 exports.SecureContext = SecureContext;