Skip to content

getparam: switch off CONNECT headers when using -J - #22838

Closed
bagder wants to merge 2 commits into
masterfrom
bagder/connect-poison
Closed

bagder wants to merge 2 commits into
masterfrom
bagder/connect-poison

Conversation

@bagder

@bagder bagder commented Sep 5, 2026

Copy link
Copy Markdown
Member

To prevent it getting trigged by proxies

Add test4656 to verify

Reported-by: Stanislav Fort

To prevent it getting trigged by proxies

Add test4656 to verify

Reported-by: Stanislav Fort
@bagder

bagder commented Sep 5, 2026

Copy link
Copy Markdown
Member Author

It is a little amusing that this test has a problem with the TCP connection closing without TLS noticing so it returns error, for some. But both AWS-LC and boring get totally confused and say the error is:

 13:42:57.809412 * TLS connect error: error:00000000:invalid library (0):OPENSSL_internal:invalid library (0)

OpenSSL gets it right though. Using the exact same API calls...

@bagder
bagder force-pushed the bagder/connect-poison branch from 5465242 to 7523865 Compare September 5, 2026 14:40
@bagder

bagder commented Sep 5, 2026

Copy link
Copy Markdown
Member Author

Our test suite has a hard time to actually do what I want here without triggering this error all over: which is to forcibly close the connection after the first request and do a new connection and new CONNECT for the second.

@bagder
bagder marked this pull request as ready for review September 5, 2026 14:59
@bagder bagder closed this in 1f2b39c Sep 5, 2026
@bagder
bagder deleted the bagder/connect-poison branch September 5, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant