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

enable more tests for hyper #7889

Closed
wants to merge 6 commits into from
Closed

enable more tests for hyper #7889

wants to merge 6 commits into from

Conversation

@bagder
Copy link
Member

@bagder bagder commented Oct 21, 2021

No description provided.

bagder added 2 commits Oct 21, 2021
It returns HYPERE_UNEXPECTED_EOF for this case which we convert to the
somewhat generic CURLE_RECV_ERROR.
bagder added 2 commits Oct 21, 2021
This test verifies that curl works with binary zeroes in HTTP response
headers and hyper refuses such. They're not kosher http.
@bagder bagder force-pushed the bagder/hyper-tests branch from 467f47c to 4c6e7db Oct 21, 2021
bagder added 2 commits Oct 22, 2021
Simply because hyper doesn't have this ability. Mentioned in docs now.

Skip test 326 then
... and make sure to stop ignoring the body once the CONNECT is done.

This should make test 206 work proper again and not be flaky.
@bagder bagder closed this in 7b4d44e Oct 22, 2021
bagder added a commit that referenced this issue Oct 22, 2021
bagder added a commit that referenced this issue Oct 22, 2021
bagder added a commit that referenced this issue Oct 22, 2021
This test verifies that curl works with binary zeroes in HTTP response
headers and hyper refuses such. They're not kosher http.

Closes #7889
bagder added a commit that referenced this issue Oct 22, 2021
Simply because hyper doesn't have this ability. Mentioned in docs now.

Skip test 326 then

Closes #7889
bagder added a commit that referenced this issue Oct 22, 2021
... and make sure to stop ignoring the body once the CONNECT is done.

This should make test 206 work proper again and not be flaky.

Closes #7889
@bagder bagder deleted the bagder/hyper-tests branch Oct 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants