codeminders / php-amqplib

PHP library implementing Advanced Message Queuing Protocol (AMQP).
GNU Lesser General Public License v2.1
0 stars 0 forks source link

authentication failure causes hang #9

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
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.

Original issue reported on code.google.com by mixedpu...@gmail.com on 29 Oct 2009 at 10:56

Attachments:

GoogleCodeExporter commented 9 years ago
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.

Original comment by mixedpu...@gmail.com on 29 Oct 2009 at 11:25

GoogleCodeExporter commented 9 years ago
Lyolik please review these bugs.

Original comment by kroko...@gmail.com on 21 Mar 2010 at 9:17