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

fread would hang indefinitely when socket was empty, this prevents that. #86

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

JohnyDays
Copy link

No description provided.

@hepspl
Copy link

hepspl commented May 10, 2015

Thank you, man! It fixed my broken cron which had been broken for weeks because of that issue.

@JohnyDays
Copy link
Author

Exact same thing happened to me, glad to help!
After reading through the issues, it seems like it's a php bug (or behaviour change) introduced in some of the latest versions, where fread waits for the specified number of bytes to arrive in the socket and they never do, whereas before it wouldnt, and this is only a temporary fix and may even cause some side effects. Regardless, it fixed it in my use case and everything seems to be working

@rwforsythe
Copy link

This seems to temporarily solve the problem, but any errors that would normally be bounced immediately are not reported. Instead they report as successfully sent.

@niclashedam
Copy link

THANK YOU!

@rwforsythe
Copy link

Note that this issue should have been fixed with newer versions of PHP: see #84

@niclashedam
Copy link

@rwforsythe Yea, I know.
But my application runs on GAE, which means I have no control over the PHP version. It could take months before Google upgrades from PHP 5.5.23

@Zaeembinrehman
Copy link

@JohnyDays
I have got the same issue but adding the 4 lines of this commit didn't solve issue for me.

PHP Version is as follows:

php git

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.

5 participants