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

fwrite infinite loop on broken pipe #50

Closed
khughitt opened this issue Apr 3, 2012 · 8 comments
Closed

fwrite infinite loop on broken pipe #50

khughitt opened this issue Apr 3, 2012 · 8 comments

Comments

@khughitt
Copy link

khughitt commented Apr 3, 2012

In an application I have where tasks may take up to 5-6 minutes the workers end up going in infinite loops with an error message to the effect of:

PHP Notice: fwrite(): send of 898 bytes failed with errno=32 Broken pipe in /var/www/lib/Redisent/Redisent.php on line 74

It looks like the same issue was reported upstream and a pull request submitted (jdp/redisent#10).

@ghost
Copy link

ghost commented Jun 18, 2012

Hi,

We are facing the same issue.
PHP Notice: fwrite(): send of 898 bytes failed with errno=32 Broken pipe in /var/www/lib/Redisent/Redisent.php on line 74
It stuck in infinite loop. What should be the proper fix for this bug. is this issue already fixed in any of the branch. Please let us know. Thanks

@khughitt
Copy link
Author

I applied the upstream fix manually and that did the trick.

@ghost
Copy link

ghost commented Jun 18, 2012

Thanks for your reply.

Question for developers/maintainers - any idea when this fix will be officially available or should we go ahead and apply the fix ourselves.

Thanks,
Nitin

@chrisboulton
Copy link
Owner

Hi,

The fix was actually pulled in a few days ago (sorry about the delay), but as of a few minutes ago, Redisent has actually been replaced by Credis which also correctly handles this.

@jadjoubran
Copy link

Can we get details on how to apply the upstream fix? or which files were changed and on which branch to fix this issue.
Thank you

@danhunsaker
Copy link
Contributor

Use a more recent version of Resque which relies on Credis instead of Redisent.

@3nr1c
Copy link

3nr1c commented Jul 7, 2016

Hi, I know this should be fixed from a long time now, but I'm getting the exact same notice. I had to change my composer.json version to "dev-master".

Please consider tagging a new version with the most recent code, as using the dev-master can be a bit insecure if somebody wants to use this in production.

@danhunsaker
Copy link
Contributor

At this point, your best bet might be to use my fork instead... Registered on Packagist under danhunsaker/php-resque.

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

No branches or pull requests

5 participants