-
Notifications
You must be signed in to change notification settings - Fork 128
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
How to disable heartbeat? #175
Comments
When a heartbeat value of 0 is set, that should be sent to the server, not the server suggested value. |
Here's the problem: https://github.com/squaremo/amqp.node/blob/6ee18f27e31863f5256bc921a21f36ebf2ba5e74/lib/connection.js#L192 You can close the issue here and open one in amqplib instead |
Thanx @carlhoerberg for the reply. If you see the rabbot code at the line You will find that if we give heartbeat: 0 while making connection even though this library is sending the default value of the heartbeat. |
@carlhoerberg, Thanks for creating the issue. But I think this rabbot library is too doing the same thing. When we set the hearbeat: 0, it sets the heartbeat to default i.e 30 seconds. This seems to be a bug. Please verify it. Please check these links https://github.com/arobson/rabbot/blob/master/src/amqp/connection.js#L46 |
no, that seems fine. it sets it to 30s if you haven't specified heartbeats at all (i which case i would use the server's suggested value, but at least that's not hte problem here) |
Cool. Lets see if this amqp-node/amqplib#467 solves the problem. On that note I'm closing this issue. Thanx! |
Hi,
We need to disable the heartbeat as suggested by CloudAMQ Support. Kindly suggest how can we do this?
We are using the following versions of rabbot client and amqp server.
client:
[email protected]
Server:
[email protected]
[email protected]
Note: We are using CloudAMQ as SAAS.
Thanks.
The text was updated successfully, but these errors were encountered: