Again Websocket Reconnection about coinbase-pro-node HOT 6 CLOSED

@bennyn thanks a lot. I will checkout the new version later sometime. For now I am closing the issue.

from coinbase-pro-node.

Thanks for reporting. It looks reconnecting-websocket (the library which I use) is trying to do something smart. Have you checked what happens when you adjust the connectionTimeout given to the WebSocketClient.connect method?

from coinbase-pro-node.

Yes, I tried, 4, 8 and 10 seconds but result is the same. This is why I asked you to provide arguments for the connect method.

from coinbase-pro-node.

@bennyn , I ended up implementing my own websocket tracker. I believe it is not the best one, but it has been running for more than 6 hours and there is NO single interruption, though I implemented reconnection mechanism as well. Check it out here.

Once you solve that issue and update your repo please let me know. Looks like the main problem is the library you use.

Feel free to close the issue if you think it is not relevant anymore.

from coinbase-pro-node.

Thanks for you input. I pushed big improvements to the WebSocket implementation of "coinbase-pro-node" today. These are available in v1.6.1.

I also updated the WebSocket example to describe the full lifecycle (connect → subscribe → unsubscribe → disconnect). There now is a demo script which is executable and connects to the ticker to test the lifecycle. When removing the disconnecting part, then I am able to run it without interruptions. It even reconnects when switching my computer to "Airplane mode" and back again. ✈

Looking at your previous code I noticed that you subscribed to the channel without waiting for the open event. That can cause a lot of troubles since you may send subscribe messages to a channel which is not open to receive messages. Your current code waits for "open" before sending messages, so that's good. 👍

from coinbase-pro-node.

Cool. I sent you a pull request which should make testing easier: https://github.com/baloian/cetracker/pull/4

from coinbase-pro-node.