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

Jump to bottom

curl_easy_header fails when requesting CURLH_PSEUDO headers #9235

Closed
charonn0 opened this issue Jul 31, 2022 · 1 comment
Closed

curl_easy_header fails when requesting CURLH_PSEUDO headers #9235

charonn0 opened this issue Jul 31, 2022 · 1 comment
Labels
HTTP libcurl API

Comments

@charonn0
Copy link
Contributor

I did this

Call curl_easy_header with the CURLH_PSEUDO origin, either alone or in a bitmask of other origins.

I expected the following

To receive a pointer to the header struct. Instead libcurl returns CURLHE_BAD_ARGUMENT. It looks like pseudo headers were omitted from a sanity check: https://github.com/curl/curl/blob/master/lib/headers.c#L77

curl/libcurl version

libcurl/7.84.0 OpenSSL/3.0.4 (Schannel) zlib/1.2.12 brotli/1.0.9 libidn2/2.3.2 libssh2/1.10.0 nghttp2/1.48.0 ngtcp2/0.6.0 nghttp3/0.5.0 libgsasl/1.10.0

operating system

Windows 10 x64
@bagder
Copy link
Member

bagder commented Jul 31, 2022

Seems entirely correct. Feel free to submit a PR or I will do it soon.

@charonn0 charonn0 mentioned this issue Jul 31, 2022
Closed
@bagder bagder closed this as completed in 4641575 Aug 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
HTTP libcurl API
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

curl_easy_header: Add CURLH_PSEUDO to sanity check
2 participants
@bagder @charonn0