search

OpenZeppelin / openzeppelin-gsn-provider

Web3 provider for the Gas Station Network
MIT License
39 stars 29 forks source link

Better error message on failed canRelay #4

Closed spalladino closed 5 years ago

spalladino commented 5 years ago

If a sendTransaction fails because canRelay fails, the error bubbled to the client is "no relay is available to do this operation", which makes it difficult to debug.

The error should be more informative explaining that some relays were reached, but they refused to execute because of canRelay.

nventuro commented 5 years ago

The error message should also be stable (at least a part of it), so that it can be detected by test helpers such as expectRevert (expectGSNRejected?)