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.

Sign up for GitHub

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

Jump to bottom

Fix bug with not setting Sec-WebSocket-Key header. #964

Merged

ysmood merged 1 commit into go-rod:main from pydima:main

Oct 26, 2023

Contributor

pydima commented Oct 26, 2023

Development guide

Test on local before making the PR

go run ./lib/utils/simple-check

Sorry, made a bug in previous PR.


          Fix bug with not setting Sec-WebSocket-Key header.

cbb5612

Contributor Author

pydima commented Oct 26, 2023

Hmm, seems like pipeline failure is not relevant to the change.

ysmood merged commit 3522cb9 into go-rod:main

5 of 6 checks passed

Member

ysmood commented Oct 26, 2023

Yes, the network is not stable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment