Skip to content

url: set default upload flags to CURLULFLAG_SEEN#21217

Closed
bagder wants to merge 1 commit intomasterfrom
bagder/default-upload-flags
Closed

url: set default upload flags to CURLULFLAG_SEEN#21217
bagder wants to merge 1 commit intomasterfrom
bagder/default-upload-flags

Conversation

@bagder
Copy link
Copy Markdown
Member

@bagder bagder commented Apr 4, 2026

As documented. Not doing this was a regression.

Follow-up to 6758aa7

Reported by Codex Security.

As documented. Not doing this was a regression.

Follow-up to 6758aa7

Reported by Codex Security.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Sets libcurl’s default value for CURLOPT_UPLOAD_FLAGS to CURLULFLAG_SEEN, aligning the library default with the documented behavior and fixing a reported regression.

Changes:

  • Initialize UserDefined::upload_flags to CURLULFLAG_SEEN in Curl_init_userdefined().

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

@bagder bagder closed this in 6d6d81e Apr 4, 2026
@bagder bagder deleted the bagder/default-upload-flags branch April 4, 2026 15:07
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