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

tests, increase sws timeout for more robust testing #10898

Closed
wants to merge 1 commit into from

Conversation

icing
Copy link
Contributor

@icing icing commented Apr 6, 2023

  • for https CONNECT forwarding, this was fixed at 5 seconds which led to spurious CI test failures
  • add --keepalive parameter to sws to control this
  • let httpserver use 30 seconds

- for https CONNECT forwarding, this was fixed at 5 seconds
  which led to spurious CI test failures
- add --keepalive parameter to sws to control this
- let httpserver use 30 seconds
@github-actions github-actions bot added the tests label Apr 6, 2023
@bagder bagder closed this in 356dd0b Apr 6, 2023
@bagder
Copy link
Member

bagder commented Apr 6, 2023

Thanks!

bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
- for https CONNECT forwarding, this was fixed at 5 seconds
  which led to spurious CI test failures
- add --keepalive parameter to sws to control this
- let httpserver use 30 seconds

Closes curl#10898
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

2 participants