search

cosmos / cosmjs

The Swiss Army knife to power JavaScript based client solutions ranging from Web apps/explorers over browser extensions to server-side clients like faucets/scrapers.
https://cosmos.github.io/cosmjs/
Apache License 2.0
657 stars 343 forks source link

@cosmjs/tendermint-rpc - Suddenly getting error after broadcastTx->decodeTx "Error: Invalid string. Length must be a multiple of 4" #1517

Closed chasevoorhees closed 11 months ago

chasevoorhees commented 11 months ago

image "@cosmjs/tendermint-rpc": "^0.31.3", "@cosmjs/tendermint-rpc": "^0.32.1",

This just started happening a couple days ago. Have tested with a few versions now.

The transaction signs/broadcasts successfully, then this happens when the TX is parsed back in Going through the events: fromBase64('spender') ends up getting called with the event key 'spender' for some reason.

This is the offending file: node_modules/osmojs/node_modules/@cosmjs/tendermint-rpc/build/tendermint34/adaptor/responses.js

image image

I'll look into this more tomorrow but hope someone else has encountered this/might have an idea.

webmaster128 commented 11 months ago

Osmosis testnet was upgraded to Cosmos SDK 0.47 as far as I can tell. In that case, using the Tendermint34Client won't work anymore. You need the Tendermint37Client. Can you report this to osmojs where those things are defined?

chasevoorhees commented 11 months ago

Osmosis testnet was upgraded to Cosmos SDK 0.47 as far as I can tell. In that case, using the Tendermint34Client won't work anymore. You need the Tendermint37Client. Can you report this to osmojs where those things are defined?

Thanks for this. Does this also apply to Osmosis mainnet, or is there a date set for that?

I've opened an issue with their repo

webmaster128 commented 11 months ago

Let's further discuss this on the Osmosis repo. Cheers!