Broadcast timeout should still display the TxHash

[greg-szabo]

When broadcasting a transaction, the binary might fail to wait for the transaction to be included in a block with the error:

Error: RPC error -32603 - Internal error: timed out waiting for tx to be included in a block

BUT the transaction hash is already available so it should be printed so the user can troubleshoot the issue further using other methods.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged
• Contributor assigned/self-assigned

[alexanderbez]

This is fair, but note, broadcast mode block is deprecated and should not really be used at all.

[greg-szabo]

The error message comes from execution with the default broadcast mode of sync.

But regardless of the broadcast mode, the TxHash should be returned, I think. It would really help troubleshooting for the end-user.

[alexanderbez]

This doesn't doesn't sound right. If you see the "timed out waiting for tx to be included in a block" error message, that means block mode was used, indicating that sync is not the real default.

What app and version of the app were you using?

[greg-szabo]

Hm... I think I might remember incorrectly and it might have been block. I'll double-check. It was either SDK 0.44 or 0.45 (possibly gaiad and/or osmosisd.)

[greg-szabo]

The default on recent binaries is "sync" both in gaiad and osmosisd. I'll close this and keep track of #12167 .