Skip to content

/curl

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

HTTP headers are delivered in pieces #2314

Closed
bagder opened this issue Feb 15, 2018 · 1 comment

Comments

Assignees
No one assigned
Labels
HTTP
Projects
None yet
Milestone
No milestone
2 participants
@bagder @jay
@bagder
Copy link
Member

commented Feb 15, 2018

I did this

Get a HTTP response with one or more headers longer than 16K. Get the headers sent to the header callback in 16K pieces.

I expected the following

To get the full headers delivered to the CURLOPT_HEADERFUNCTION as per the documentation.

curl/libcurl version

current git master

operating system

any

Reported-by: Guido Berhoerster
Bug: https://curl.haxx.se/mail/lib-2018-02/0056.html

@bagder bagder added the HTTP label Feb 15, 2018

@jay

This comment has been minimized.

Sign in to view
Copy link
Member

commented Feb 15, 2018

See #801 which went stale when I moved on to other things. There might be something salvageable in there

bagder added a commit that referenced this issue Feb 16, 2018

@bagder
header callback: don't chop headers into smaller pieces 
Reported-by: Guido Berhoerster
Fixes #2314
Verified
This commit was signed with a verified signature.
@bagder bagder Daniel Stenberg
GPG key ID: 5CC908FDB71E12C2 Learn about signing commits
3349ebe

@bagder bagder closed this in 367689d Feb 16, 2018

fsedano added a commit to fsedano/curl that referenced this issue Feb 20, 2018

@bagder @fsedano
header callback: don't chop headers into smaller pieces 
Reported-by: Guido Berhoerster
Fixes curl#2314
Closes curl#2316
7b553ef

@lock lock bot locked as resolved and limited conversation to collaborators May 17, 2018

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.