Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problem: Tx decoder rejects unknown transaction types by default #681

Open
calvinaco opened this issue Feb 4, 2022 · 0 comments
Open

Problem: Tx decoder rejects unknown transaction types by default #681

calvinaco opened this issue Feb 4, 2022 · 0 comments

Comments

@calvinaco
Copy link
Collaborator

calvinaco commented Feb 4, 2022

Background

Each Cosmos-based chains may have included different modules onto the chain. In the current design the default behaviour upon unknown transaction types is to panic, which is good for development but produce unfriendly result to production.

Proposed Solution

  • Provide default RegisterInterface for different chains (chains can also submit new PR to add their own Interfaces)
  • Offer configs on decode error behaviour (panic or ignore)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant