This repository has been archived by the owner on Feb 26, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 686
VM Revert Reasons not returned #480
Labels
Comments
This seems to be a problem in estimate gas. Investigating further |
Previously, estimate gas was incorrectly returning estimates for failing transactions instead of failing (like other clients do). We'll look into providing the revert reason string to the error message in estimate gas. /cc @nicholasjpaterno |
davidmurdoch
added a commit
that referenced
this issue
Nov 6, 2019
davidmurdoch
added a commit
that referenced
this issue
Nov 25, 2019
Is there a plan to release #502 any soon? |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Revert reasons are not being passed back by ganache-core anymore
Expected Behavior
Old error.message (v6.4.1)
Returned error: VM Exception while processing transaction: revert ${solidityRevertReason}
Current Behavior
error.message being passed back (v6.7.0-beta.0)
error: execution error: revert
Possible Solution
Steps to Reproduce (for bugs)
Send a transaction that reverts with a solidity revert string.
Your Environment
The text was updated successfully, but these errors were encountered: