-
Notifications
You must be signed in to change notification settings - Fork 71
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
Add context to contract exceptions #1743
Comments
YaroShkvorets
added a commit
to YaroShkvorets/leap
that referenced
this issue
Oct 10, 2023
heifner
added a commit
that referenced
this issue
Oct 11, 2023
…ceptions Add action context to contract error message
YaroShkvorets
added a commit
to YaroShkvorets/leap
that referenced
this issue
Nov 14, 2023
heifner
added a commit
that referenced
this issue
Nov 15, 2023
…ntract-exceptions [5.0] Add context to contract exceptions
heifner
added a commit
that referenced
this issue
Nov 15, 2023
…act-exceptions-main [5.0 -> main] Add context to contract exceptions
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Problem
There is that annoying problem with errors coming from smart contracts.
When your smart contract makes inline action calls to external contracts and one of these actions fails, you just get an error with zero context, so you can’t tell which smart contract and action that error came from. You can probably recognize your own errors but not some random errors coming from other contracts.
I.e. you only get this:
Assertion "check" failures also have this problem:
This makes debugging quite painful.
Proposal
Add additional context to the error message:
The text was updated successfully, but these errors were encountered: