[js] Check if proxy is enabled and allow selenium manager to use proxy options

harsha509 · harsha509 · commit a6c7e03f4ad3 · 2023-06-06T20:27:29.000+05:30
diff --git a/javascript/node/selenium-webdriver/common/seleniumManager.js b/javascript/node/selenium-webdriver/common/seleniumManager.js
@@ -89,14 +89,20 @@ function driverLocation(options) {
     args.push('--browser-path', '"' + vendorOptions.binary + '"')
   }
 
-  // Get the httpProxy and sslProxy values from the options object
-  const httpProxy = options.getProxy()['httpProxy'];
-  const sslProxy = options.getProxy()['sslProxy'];
-
-  if (httpProxy) {
-    args.push('--proxy', httpProxy);
-  } else if (sslProxy) {
-    args.push('--proxy', sslProxy);
+  const proxyOptions = options.getProxy();
+
+  // Check if proxyOptions exists and has properties
+  if (proxyOptions && Object.keys(proxyOptions).length > 0) {
+    const httpProxy = proxyOptions['httpProxy'];
+    const sslProxy = proxyOptions['sslProxy'];
+
+    if (httpProxy !== undefined) {
+      args.push('--proxy', httpProxy);
+    }
+
+    else if (sslProxy !== undefined) {
+      args.push('--proxy', sslProxy);
+    }
   }
 
   const smBinary = getBinary()
