Skip to content
This repository has been archived by the owner on Jul 28, 2022. It is now read-only.

Adding GSN support #1

Open
wants to merge 1 commit into
base: workshop
Choose a base branch
from
Open

Adding GSN support #1

wants to merge 1 commit into from

Conversation

drortirosh
Copy link
Member

add GSN provider support

minimal set of changes to add GSN support to the contract and the dApp
so that calling user no longer needs eth.

Actual payment is done by the default "accept everything" paymaster
contract.

add GSN provider support

minimal set of changes to add GSN support to the contract and the dApp
so that calling user no longer needs eth.

Actual payment is done by the default "accept everything" paymaster
contract.
@ghost
Copy link

ghost commented Nov 17, 2021

Hi, while I am doing with GSN getting this kind of error
Relaying errors (1):
http://127.0.0.1:65432 => paymaster rejected in local view call to 'relayCall()' : FWD: signature mismatch

@drortirosh
Copy link
Member Author

Hi, while I am doing with GSN getting this kind of error Relaying errors (1): http://127.0.0.1:65432 => paymaster rejected in local view call to 'relayCall()' : FWD: signature mismatch

please try to follow the https://docs.opengsn.org/faq, as I think there is such an issue there.
If not, please use our https://forum.opengsn.org to ask more questions, as the "issues" is less appropriate for them.

Copy link

@Tranquil-Flow Tranquil-Flow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In CaptureTheFlag.sol, the implementation for setting the trustedForwarder on L15 is no longer correct. Suggest using:

_setTrustedForwarder(forwarder);

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants