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

StreamConnection does not time out #408

Closed
prolic opened this issue Jun 3, 2016 · 3 comments
Closed

StreamConnection does not time out #408

prolic opened this issue Jun 3, 2016 · 3 comments
Assignees
Milestone

Comments

@prolic
Copy link
Contributor

prolic commented Jun 3, 2016

I have the following scenario:

Push 5 messages to a queue, and try to consume 10.

When using php amqp extension, I receive an exception like: "Consumer timeout exceed".

When using php-amqplib SocketConnection, I receive an exception like: "Error reading data. Received 0 instead of expected 7 bytes". I can catch that and turn into a more readable exception.

When using php-amqplib StreamConnecton, I do not receive any Exception, the consumer just hangs forever, which is highly undesired.

@Xedecimal
Copy link

I believe the issue you are having is in StreamIO::read($len) that says

$this->select(null, null);

This goes down to

stream_select($read, $write, $except, null, null);

Finally you'll see that

image

and voila, frozen processes for all!

@lukebakken
Copy link
Collaborator

Please see #423. If you could test out that patch, it would be very helpful. Thanks!

@lukebakken lukebakken assigned lukebakken and unassigned lukebakken Jan 16, 2018
@lukebakken lukebakken modified the milestones: 2.7.1, 2.7.2 Jan 29, 2018
@lukebakken lukebakken removed this from the 2.7.2 milestone Feb 10, 2018
@lukebakken
Copy link
Collaborator

Assuming this has been addressed by #423 and will ship in 2.8.0. Please test an RC that will be available this week.

@lukebakken lukebakken added this to the 2.8.0 milestone Oct 9, 2018
@lukebakken lukebakken self-assigned this Oct 9, 2018
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

3 participants