BuggyShow-transfers Query, doesn't yield anything, cli hangs and sometimes freezes the node

[spidey-169]

The show-transfer query parameter, which is very useful tool to get the list of confirmed transfers from a wallet, is not yielding anything but simply freezes the node when executed.

The commands I have tried

namadac show-transfers --node http://127.0.0.1:26657 --owner $NEW_WALLET --token tnam1qxvg64psvhwumv3mwrrjfcz0h3t3274hwggyzcee

namadac show-transfers --node <RPC> --owner $NEW_WALLET --token tnam1qxvg64psvhwumv3mwrrjfcz0h3t3274hwggyzcee

For this testing, I created a new wallet and just had 1 transaction using it.However, even with that the query freezes and doesn't yield anything. Bringing it forward so it can be fixed as this is a very useful query tool to have.! @brentstone Tagging you here. Thanks

[spidey-169]

More details on this issue. Looks like the client goes into an infinite loop for Converting current asset type to latest asset type. Note that this command was executed for a new wallet with a single transaction.

[brentstone]

@Fraccaman @grarco ?

[spidey-169]

@Fraccaman @grarco ?

Adding to the debug studies, for some wallets ("new-wallet" wallet), instead of going into an infinite converting current asset type to latest asset type if gives application panic crashes error

while for others ("test" wallet), it still goes into an "infinte" converting loop. Tested on two different nodes and giving different results

[spidey-169]

When checking the balance (new-wallet), the screen shows balance and for some reason, CLI is trying to query the balance over a number of IBC channels? Could this be why its going into anor just Application Panic crashing? I only see this screen for the case where "Application panic crashed error" appeared

For the test wallet where it went into a infinite loop of for "Changing current asset type >>>" the balance shows up without these error messages (test wallet)

[spidey-169]

Apologies for tagging again, @brentstone Any thoughts on this when you get a chance to look into this? Thanks 🙂

[grarco]

This is related to #2381 since internally show-transfers calls query_tx_deltas

[spidey-169]

This is related to #2381 since internally show-transfers calls query_tx_deltas

Great. Thanks for confirming the bug.

[cwgoes]

Thanks for your report! We're actually going to remove this command, and replace it later with a faster option using an indexer.