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

feature request: command to display non-parametric error message as original text instead of unicode string of hash for reverted transactions on verified smart contracts #229

Open
bguiz opened this issue Feb 13, 2024 · 0 comments

Comments

@bguiz
Copy link

bguiz commented Feb 13, 2024

What:

  • use this reverted transaction as an example: https://hashscan.io/testnet/transaction/1707310800.715788003
  • it is displayed as 3�J --> convert to hexadecimal 0x33984a19 ---> which is a hash of DuplicateCluster() (run cast sig 'DuplicateCluster()' to verify it)
  • hcli command to parse the above automatically, by processing the above automatically and display DuplicateCluster() instead of 3�J
  • no need to do this for all reverted SC transactions, apply the following filters: (1) only needed for verified SCs will need to precompute hashes of errors from the ABI, and (2) only needed for errors with zero parameters, as we can tackle the parametric errors in a separate feature request that is expected to be more complex to implement

Why:

  • easier to tell at a glance why a transaction has failed
  • aids in debugging SC failures
  • reduce support volume from integration partners (this is a live example)
@bguiz bguiz converted this from a draft issue Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Community Requests
Development

No branches or pull requests

1 participant