noodlefrenzy / node-amqp10

amqp10 is a promise-based, AMQP 1.0 compliant node.js client
MIT License
134 stars 56 forks source link

fix(heartbeat): local vs remote idleTimeouts #261

Closed dnwe closed 8 years ago

dnwe commented 8 years ago

There were a couple of bugs here:

NB: currently, although we flow an idleTimeout in our OpenFrame, we don't actually monitor whether or not the remote-end actually sends us data within this interval, nor do we disconnect them if they don't.