Skip to content

ext/curl: Propagate multipart field construction errors - #400

Closed
iliaal wants to merge 1 commit into
PHP-8.4from
fix/aph-curl-mime-error-code-discarded-tlk-84-work
Closed

iliaal wants to merge 1 commit into
PHP-8.4from
fix/aph-curl-mime-error-code-discarded-tlk-84-work

Conversation

@iliaal

@iliaal iliaal commented Sep 25, 2026 •

Copy link
Copy Markdown
Owner

Errors from adding scalar and nested POST fields were ignored, allowing curl_setopt() to succeed with incomplete multipart data.

Propagate those errors through the existing cleanup path, including when rebuilding MIME data for copied or cloned handles.

Retain add_simple_field() errors for scalar and nested POST fields so
curl_setopt(), curl_copy_handle(), and cloning reject incomplete MIME data
through the existing error and cleanup paths.
@iliaal
iliaal force-pushed the fix/aph-curl-mime-error-code-discarded-tlk-84-work branch from 77ae9a5 to 5ecb97d Compare October 19, 2026 12:41
@iliaal iliaal changed the title ext/curl: Propagate MIME builder errors ext/curl: Propagate multipart field construction errors Oct 19, 2026
@iliaal

iliaal commented Oct 19, 2026

Copy link
Copy Markdown
Owner Author

Promoted to php#24434: php#24434

@iliaal iliaal closed this Oct 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant