Skip to content

ext/curl: add more options to curl - #22840

Merged
kocsismate merged 1 commit into
php:masterfrom
Sjord:curl-more-options
Nov 12, 2026
Merged

kocsismate merged 1 commit into
php:masterfrom
Sjord:curl-more-options

Conversation

@Sjord

@Sjord Sjord commented Jul 21, 2026

Copy link
Copy Markdown
Contributor
  • CURLOPT_ECH
  • CURLOPT_HAPROXY_CLIENT_IP
  • CURLOPT_SERVER_RESPONSE_TIMEOUT_MS
  • CURLOPT_MAIL_RCPT_ALLLOWFAILS is renamed to CURLOPT_MAIL_RCPT_ALLOWFAILS

@Sjord

Sjord commented Jul 21, 2026

Copy link
Copy Markdown
Contributor Author

Fixes #16305

@Sjord Sjord changed the title Add more options to curl ext/curl: add more options to curl Jul 30, 2026
@Sjord
Sjord marked this pull request as ready for review August 10, 2026 09:12
@Sjord
Sjord requested review from adoy and kocsismate as code owners August 10, 2026 09:12
@Sjord
Sjord force-pushed the curl-more-options branch 3 times, most recently from b4e724f to 959627b Compare September 6, 2026 09:10
@Sjord

Sjord commented Oct 28, 2026

Copy link
Copy Markdown
Contributor Author

@iluuu1994 Could you take a look at this?

Comment thread ext/curl/curl.stub.php Resolved
Comment thread ext/curl/curl.stub.php Resolved
Comment thread ext/curl/interface.c Resolved
Comment thread ext/curl/interface.c Resolved
@kocsismate

Copy link
Copy Markdown
Member

The arginfo file needs to be regenerated. Then we are good to go.

@Sjord
Sjord force-pushed the curl-more-options branch from 04f38f5 to 2822e4f Compare November 12, 2026 07:46
@kocsismate

Copy link
Copy Markdown
Member

@Sjord Sorry, I forgot to ask you to mention the new constants in the UPGRADING file. :(

- CURLOPT_ECH
- CURLOPT_HAPROXY_CLIENT_IP
- CURLOPT_SERVER_RESPONSE_TIMEOUT_MS
- CURLOPT_MAIL_RCPT_ALLLOWFAILS is renamed to CURLOPT_MAIL_RCPT_ALLOWFAILS
@Sjord
Sjord force-pushed the curl-more-options branch from 2822e4f to 0b6bec7 Compare November 12, 2026 09:37

@kocsismate kocsismate left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!

@kocsismate
kocsismate merged commit 93cb276 into php:master Nov 12, 2026
17 of 18 checks passed
@Sjord Sjord mentioned this pull request Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants