-
Notifications
You must be signed in to change notification settings - Fork 7
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
Handling invalid proofs #9
Comments
Well, according to the solidity documentation, "Catching exceptions is not yet possible." so maybe that influences this decision, personally I think reverting is saner, but if the EVM doesn't support catching exceptions then that leaves a developer with no real way of doing anything other than bubbling up and reverting as well, if a MPP fails |
Agreed, let's revert. We could handle exceptions with a low-level solidity "exception handler" that catches reverts and returns |
On receiving an invalid proof, the
ProvethVerifier
contract currently:It should be either one or the other.
The text was updated successfully, but these errors were encountered: