Skip to content

[cURL] [bug#77946] Add test for curl_multi_info_read() #4075

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

phansys
Copy link
Contributor

@phansys phansys commented Apr 27, 2019

cURL resources returned by curl_multi_info_read() aren't usable with curl_errno() or curl_errno(). When an error occurs with these resources, the "handle" returned doesn't behave as a single handle.

@phansys phansys force-pushed the curl_multi_info_read branch from 6b61c3f to bad1ac3 Compare April 27, 2019 12:13
@phansys phansys changed the title [cURL] Add test for curl_multi_info_read() [cURL] [bug#77946] Add test for curl_multi_info_read() Apr 27, 2019
@phansys phansys force-pushed the curl_multi_info_read branch 2 times, most recently from 4a111dd to 552a971 Compare April 29, 2019 15:54
@phansys phansys changed the base branch from master to PHP-7.2 April 29, 2019 15:54
@phansys phansys force-pushed the curl_multi_info_read branch 2 times, most recently from d19e7a9 to e423fee Compare April 29, 2019 19:34
@phansys phansys marked this pull request as ready for review April 29, 2019 19:37
@phansys
Copy link
Contributor Author

phansys commented Jul 22, 2019

Closed by c8c183e.

Thank you @valga!

@phansys phansys closed this Jul 22, 2019
@phansys phansys deleted the curl_multi_info_read branch May 23, 2020 16:56
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