Skip to content

CURLOPT_POSTFIELDSIZE.md: improve wording - #23078

Closed
bagder wants to merge 1 commit into
masterfrom
bagder/postsize-wording
Closed

bagder wants to merge 1 commit into
masterfrom
bagder/postsize-wording

Conversation

@bagder

@bagder bagder commented Sep 24, 2026

Copy link
Copy Markdown
Member

Clarify that the size must not be larger than the buffer.

Clarify that the size must not be larger than the buffer.
@bagder
bagder requested a balanced review from Copilot September 24, 2026 11:20
@github-actions github-actions Bot added the HTTP label Sep 24, 2026
@bagder
bagder marked this pull request as ready for review September 24, 2026 11:21

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.

Copilot review overview

🟢 Approval recommended

The documentation changes are accurate, consistent, and complete.

Review effort: Balanced
Findings: None

What changed in this PR

Clarifies safe use of POST field size options.

Changes:

  • Improves size-setting descriptions.
  • Warns that configured sizes must not exceed provided buffers.
File Description
docs/​libcurl/​opts/​CURLOPT_POSTFIELDSIZE.md Clarifies buffer-size requirements.
docs/​libcurl/​opts/​CURLOPT_POSTFIELDSIZE_LARGE.md Applies equivalent guidance to large sizes.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bagder bagder closed this in 6ee40bd Sep 24, 2026
@bagder
bagder deleted the bagder/postsize-wording branch September 24, 2026 12:31
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.

2 participants