Skip to content

Custom option for cURL multi handler#2287

Merged
Nyholm merged 2 commits intoguzzle:masterfrom
alexeyshockov:curl-multi-options
Oct 23, 2019
Merged

Custom option for cURL multi handler#2287
Nyholm merged 2 commits intoguzzle:masterfrom
alexeyshockov:curl-multi-options

Conversation

@alexeyshockov
Copy link
Copy Markdown
Collaborator

@alexeyshockov alexeyshockov commented Apr 15, 2019

Custom options for cURL multi handler, allowing to configure a handle from curl_multi_init(). Something like 'curl' request options, to fix #2248, for example.

Comment thread docs/faq.rst Outdated
Comment thread tests/Handler/CurlMultiHandlerTest.php
@alexeyshockov alexeyshockov force-pushed the curl-multi-options branch 2 times, most recently from 27df4df to 8010664 Compare July 9, 2019 08:09
@alexeyshockov
Copy link
Copy Markdown
Collaborator Author

So tests are failing because of the issues in master (#2327 probably should fix it). Otherwise the feature is ready.

@Gemorroj
Copy link
Copy Markdown

@sagikazarmark ping

@alexeyshockov alexeyshockov force-pushed the curl-multi-options branch 3 times, most recently from fe66dcf to 89b4a6e Compare October 14, 2019 11:49
@alexeyshockov
Copy link
Copy Markdown
Collaborator Author

@sagikazarmark, could you take a look at this for the next release?

Copy link
Copy Markdown
Member

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

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

Thank you for this PR.
I think this looks good. But I would like an other review on it.

@Nyholm Nyholm added this to the 6.5.0 milestone Oct 18, 2019
@sagikazarmark sagikazarmark self-requested a review October 18, 2019 13:18
Copy link
Copy Markdown
Member

@sagikazarmark sagikazarmark left a comment

Choose a reason for hiding this comment

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

Thanks!

@Nyholm Nyholm merged commit 8f10d0e into guzzle:master Oct 23, 2019
@alexeyshockov alexeyshockov deleted the curl-multi-options branch October 23, 2019 16:22
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.

429 - to many requests

6 participants