Skip to content
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

Update Socket to work around sending secure HTTPS requests with PHP < 7.1.4 #109

Merged
merged 2 commits into from
Sep 8, 2017

Conversation

clue
Copy link
Member

@clue clue commented Sep 8, 2017

PHP < 7.1.4 (and PHP < 7.0.18) suffers from a bug when writing big chunks of data over TLS streams at once. We try to work around this by limiting the write chunk size to 8192 bytes for older PHP versions only. This is only a work-around and has a noticable performance penalty on affected versions. Please update your PHP version.

Resolves / closes #60
Refs reactphp/socket#105

@clue clue added the bug label Sep 8, 2017
@clue clue added this to the v0.5.5 milestone Sep 8, 2017
@clue
Copy link
Member Author

clue commented Sep 8, 2017

Note that the included tests are a bit fragile and have repeatedly failed here. Restarting the builds fixed the tests, so it appears that the Travis matrix issues too many parallel requests.

I've updated the tests to be spread across eu.httpbin.org and httpbin.org at least and I'm open for suggestions 👍

@WyriHaximus
Copy link
Member

@clue I've updated the repository settings on travis to only run 1 concurrent build

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error when sending large messages over secure HTTPS with older PHP versions
3 participants