Implement network failure detection about bunny HOT 7 CLOSED

Carl,

Your understanding is correct: this suggests a TCP connection failure which Bunny 0.9 does not detect yet. This is the next big thing I am going to work on.

from bunny.

Using heartbeats should help. Can you be using load balancers/proxies by any chance? Those may be closing connections they consider idle. Bunny 0.9 should set SO_KEEPALIVE on the socket it uses.

from bunny.

do you want me to test with a LB or are you asking if i used one when i got this error? yes and no..

from bunny.

Some observations on a long-running Bunny/RabbitMQ connection in irb that you may be interested in -

1. Bunny 0.9 does not seem to set keepalive on a socket by default, however, you can explicitly set it with Bunny.new(:keepalive => true)
2. RabbitMQ 3.0.0 has a default heartbeat interval of 600 secs. After using Wireshark, the server did indeed send heartbeats to the client and kills the connection some time after 2 heartbeats have been sent without reply. This results in the loop described above.
3. Bunny::Session#negotiate_value method doesn't allow the heartbeat interval to be set to 0 if the server setting is > 0. Looking at the AMQP 0.9.1 spec (http://www.rabbitmq.com/amqp-0-9-1-reference.html), I think that Bunny should allow the heartbeat interval to be set to 0 regardless of the server setting.

from bunny.

Similar thing happens when a connection is forcefully closed:

NoMethodError
undefined method `read_fully' for nil:NilClass
/Users/carl/.rvm/gems/ruby-1.9.3-p194/gems/bunny-0.9.0.pre3/lib/bunny/transport.rb:109:in `read_next_frame'
/Users/carl/.rvm/gems/ruby-1.9.3-p194/gems/bunny-0.9.0.pre3/lib/bunny/main_loop.rb:26:in `block in run_loop'
/Users/carl/.rvm/gems/ruby-1.9.3-p194/gems/bunny-0.9.0.pre3/lib/bunny/main_loop.rb:22:in `loop'
/Users/carl/.rvm/gems/ruby-1.9.3-p194/gems/bunny-0.9.0.pre3/lib/bunny/main_loop.rb:22:in `run_loop'

from bunny.

I agree with that we probably should allow setting heartbeats to 0. I will look into that tomorrow. And we should make :keepalive => true the default, it makes much more sense that way.

from bunny.

Done, 0.9.0.pre6 includes this feature.

from bunny.