Skip to content

docs: returned header size reflects HTTP/1-style format#21912

Closed
vszakats wants to merge 4 commits into
curl:masterfrom
vszakats:docs-headersize
Closed

docs: returned header size reflects HTTP/1-style format#21912
vszakats wants to merge 4 commits into
curl:masterfrom
vszakats:docs-headersize

Conversation

@vszakats

@vszakats vszakats commented Jun 8, 2026

Copy link
Copy Markdown
Member

Ref: #21889

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the CURLINFO_HEADER_SIZE documentation to clarify that the reported header size corresponds to libcurl’s HTTP/1-style textual header representation, which is particularly relevant when using HTTP/2+ where on-the-wire header bytes differ due to framing/compression.

Changes:

  • Clarify that headers are measured in an HTTP/1-style textual format rather than raw on-the-wire bytes.
  • Add a note that HTTP/2+ wire/TLS-layer byte counts differ from the reported value.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/libcurl/opts/CURLINFO_HEADER_SIZE.md Outdated
@vszakats vszakats changed the title CURLINFO_HEADER_SIZE.md: size reflects HTTP/1 format docs: returned header size reflects HTTP/1 format Jun 8, 2026
@vszakats vszakats changed the title docs: returned header size reflects HTTP/1 format docs: returned header size reflects HTTP/1-style format Jun 8, 2026
@vszakats vszakats requested a review from Copilot June 8, 2026 18:05
@vszakats vszakats marked this pull request as ready for review June 8, 2026 18:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@vszakats vszakats closed this in b9b2c0c Jun 9, 2026
@vszakats vszakats deleted the docs-headersize branch June 9, 2026 08:24
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.

3 participants