Export to GitHub

php-amqplib - issue #9

authentication failure causes hang


Posted on Oct 29, 2009 by Massive Bird

try authenticating with bad credentials, and you'll get a hang until php times out. This is because amqp does not specify how to handle a failed authentication, and servers tend to just close the connection. php-amqplib doesn't consider that possibility. A patch to fix is attached. You don't get an error that indicates auth failure (well, how can you really?) but at least you don't have the php process hang until the 30 second timeout is hit.

Attachments

Comment #1

Posted on Oct 29, 2009 by Massive Bird

Actually, remove the check for timed_out. That breaks consumers. For the authentication failure, you only need to check for eof and break out of the loop.

Comment #2

Posted on Mar 21, 2010 by Happy Lion

Lyolik please review these bugs.

Status: New

Labels:
Type-Defect Priority-Medium