-
Notifications
You must be signed in to change notification settings - Fork 2.3k
set correct port for sauceSeleniumAddress for HTTP protocol #4961
base: master
Are you sure you want to change the base?
Conversation
when sauceSeleniumUseHttp is true, it should set the port to 80 instead of 443
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). 📝 Please visit https://cla.developers.google.com/ to sign. Once you've signed (or fixed any issues), please reply here (e.g. What to do if you already signed the CLAIndividual signers
Corporate signers
|
signed CLA |
CLAs look good, thanks! |
1 similar comment
CLAs look good, thanks! |
when sauceSeleniumUseHttp is true, it should set the port to 80 instead of 443