Merge pull request #417 from KevinGossentCap/master

other puppeteer proxy handling
vlocityinc · Nov 25, 2021 · 113be32 · 113be32
2 parents 2b4326c + 13b8723
commit 113be32
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/utilityservice.js b/lib/utilityservice.js
@@ -1508,11 +1508,11 @@ UtilityService.prototype.moveFolders = async function (source, target) {
 UtilityService.prototype.getPuppeteerOptions = async function (jobInfo) {
     let puppeteerOptions = { 
         headless: jobInfo.puppeteerHeadless,
-        args: [
-            '--no-sandbox',
-            `--proxy-server=${jobInfo.httpProxy ? jobInfo.httpProxy : ''}`
-        ]
+        args: ['--no-sandbox']
     };
+    if (jobInfo.httpProxy) {
+        puppeteerOptions.args.push('--proxy-server=' + jobInfo.httpProxy);
+    }
 
     let macChrome = path.join('/', 'Applications', 'Google Chrome.app', 'Contents', 'MacOS', 'Google Chrome');
     let winChrome = path.join('/', 'Program Files', 'Google', 'Chrome', 'Application', 'chrome.exe');