Browse Source

fix: default enable_negotiate_port to false (4-2-x) (#18287)

* fix: default enable_negotiate_port to false (#18251)

* chore: command_line is not a pointer
Jeremy Apthorp 6 years ago
parent
commit
96391dbb3b
3 changed files with 6 additions and 0 deletions
  1. 2 0
      atom/browser/io_thread.cc
  2. 3 0
      atom/common/options_switches.cc
  3. 1 0
      atom/common/options_switches.h

+ 2 - 0
atom/browser/io_thread.cc

@@ -48,6 +48,8 @@ network::mojom::HttpAuthDynamicParamsPtr CreateHttpAuthDynamicParams(
       command_line.GetSwitchValueASCII(switches::kAuthServerWhitelist);
   auth_dynamic_params->delegate_whitelist = command_line.GetSwitchValueASCII(
       switches::kAuthNegotiateDelegateWhitelist);
+  auth_dynamic_params->enable_negotiate_port =
+      command_line.HasSwitch(atom::switches::kEnableAuthNegotiatePort);
 
   return auth_dynamic_params;
 }

+ 3 - 0
atom/common/options_switches.cc

@@ -247,6 +247,9 @@ const char kAuthServerWhitelist[] = "auth-server-whitelist";
 const char kAuthNegotiateDelegateWhitelist[] =
     "auth-negotiate-delegate-whitelist";
 
+// If set, include the port in generated Kerberos SPNs.
+const char kEnableAuthNegotiatePort[] = "enable-auth-negotiate-port";
+
 }  // namespace switches
 
 }  // namespace atom

+ 1 - 0
atom/common/options_switches.h

@@ -118,6 +118,7 @@ extern const char kDiskCacheSize[];
 extern const char kIgnoreConnectionsLimit[];
 extern const char kAuthServerWhitelist[];
 extern const char kAuthNegotiateDelegateWhitelist[];
+extern const char kEnableAuthNegotiatePort[];
 
 }  // namespace switches