Websocket Bad Handshake Error about tenderduty HOT 4 CLOSED

This is an issue with the Tendermint RPC client, and it's not likely to be fixed anytime soon without an upstream patch. I've spent hours trying to track down what's going on in the code. The wss support in the client is incomplete and somehow it messes up the Dialer it hands off to Gorilla. There are probably other issues beyond that. I've had a difficult time trying to debug it. Maybe the correct approach would be to skip using the TM client entirely and just use Gorilla. I'll have to think about how difficult that would be, probably not a horribly big refactor.

from tenderduty.

Good news on this, for v2 tenderduty will use gorilla directly, and not use the tendermint client for websocket connections. This is working on the pre-alpha version, but going to leave the ticket open until there is a beta release.

from tenderduty.

Fantastic tool! Already running from Releasev2 branch.

The tool isn't connecting to the next RPC endpoint when multiple are given, and basically is stuck with the one failing one.

In my case, I had used

      - url: https://cosmos-rpc.polkachu.com:443
        alert_if_down: no
      - url: https://rpc.cosmos.network:443
        alert_if_down: no

But it doesn't even get to the second rpc server, the below error keeps looping and I see 0/2 connected in dashboard,

6:20:41 PM - could not dial ws client to wss://cosmos-rpc.polkachu.com:443/websocket: websocket: bad handshake
6:20:41 PM - ⚙️ found cosmosvaloper1sjllsnramtg3ewxqwwrwjxfgc4n4ef9u2lcnj0 (🐠stake.fish) in validator set
...

But when removing the polkachu rpc, it connects to the rpc.cosmos.network and works happily.

What's also scary is it doesn't send out any alerts either! The could be considered another bug. Maybe because it also restarts in between?

To summarize:

• Alerting needs to account for such edge case bugs too
• Try next endpoint instead of looping/restarting on websocket issue

Thanks for the great work!

from tenderduty.

Tenderduty v2 now uses the gorilla websocket client instead of tendermint's. TLS now works

from tenderduty.