Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the proxy protocol when a webdriver proxy is configured #7511

Merged
merged 1 commit into from
Sep 10, 2019

Conversation

theredfish
Copy link
Contributor

When the http protocol is different between the remote selenium
server and the webdriver proxy, an EPROTO error is encountered.

See #7509 for more information and a reproductible test case.

Could you help me to test this case correctly? It seems that changing the protocol from http to https in httpserver.js break the tests in http_test.js. Should i create a https version of Server and use it in a test?

When the http protocol is different between the remote selenium
server and the webdriver proxy, an EPROTO error is encountered.

See SeleniumHQ#7509 for more information and a reproductible test case.
@shs96c shs96c merged commit f25d117 into SeleniumHQ:master Sep 10, 2019
@shs96c
Copy link
Member

shs96c commented Sep 10, 2019

Thank you for taking the time to contribute back! It's very much appreciated! Welcome to the project!

@theredfish
Copy link
Contributor Author

You're welcome! We use Selenium everyday at Oui.sncf so it's with pleasure :)

Thank you for the merge!

jimevans pushed a commit to jimevans/selenium that referenced this pull request Sep 13, 2019
…HQ#7511)

When the http protocol is different between the remote selenium
server and the webdriver proxy, an EPROTO error is encountered.

See SeleniumHQ#7509 for more information and a reproductible test case.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants