You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Contract errors cannot be consumed programmatically from client side. A generic error is returned for failed contract invocations without the panic reason: E Input: 'Client failed with code 40: Contract error.'
Describe the solution you'd like
Output the panic reason (felt) not only to the log but also as part of the API output. For example, this is printed out to the log but not returned to the client:
I'm not sure if it is part of the JSON-RPC spec to include the errors reason in the API response? or does it break the spec if it is included? ideally, we don't want to introduce anything that's not gonna be spec compatible.
Is your feature request related to a problem? Please describe.
Contract errors cannot be consumed programmatically from client side. A generic error is returned for failed contract invocations without the panic reason:
E Input: 'Client failed with code 40: Contract error.'
Describe the solution you'd like
Output the panic reason (felt) not only to the log but also as part of the API output. For example, this is printed out to the log but not returned to the client:
The text was updated successfully, but these errors were encountered: