-
Notifications
You must be signed in to change notification settings - Fork 30
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
heartbeat troubles #26
Comments
perfect I was able to reproduce this, ill have a fix for you to try shortly |
@vkfont can you give this branch a try https://github.com/dropbox/amqp-coffee/tree/feature/0.1.21 Pending your approval ill merge it in. Thanks for the report! |
Seems to be working. At least i cant repeat this bug. |
Also found error with this code https://gist.github.com/vkfont/65d77cf461c075ebba60 on branch feature/0.1.21:
Output is:
Expected behavior: script should work |
Potentially, the problem is the queue you're declaring is auto deleted, and the consumer is trying to consume a queue that doesn't exist when it reconnects. You can catch this error from the consumer like this. You could disable auto reconnect on the consuming connection, and on disconnect set everything back up. The connection would emit a close event. It also emits a close event before it attempts to reconnect. If you have any idea's on how these errors should be handled I would love to hear them :) |
In my case setting 'autoDelete' to But if i'll set autoDelete = false, durable = true and restart rabbitmq - named channel stop receiving messages (or publisher stops send?) and no errors raised. This behaviour does not correspond to documentation: https://gist.github.com/vkfont/65d77cf461c075ebba60
UPDATE: looks my code was incorrect (wrong callback binding in .publish), everything works fine |
glad to hear it... ill probably land this on master today or tomorrow |
I have same bug described in #23
Connection is closing after [heartbeat * 3] ms
Here is code which can reproduce bug on my localhost rabbit (mac) and server rabbit (ubuntu:
https://gist.github.com/vkfont/65d77cf461c075ebba60
The text was updated successfully, but these errors were encountered: