Skip to content

using transfer to return refund to relayer? #54

Open
Amxx opened this issue May 15, 2020 · 0 comments
Open

using transfer to return refund to relayer? #54

Amxx opened this issue May 15, 2020 · 0 comments

Comments

@Amxx
Copy link

Amxx commented May 15, 2020

In ERC20Tornado's implementation, refund is forwarded to the recipient using _recipient.call or, if that fails, returned to the relayer using _relayer.transfer

see:

_relayer.transfer(_refund);

I believe this will limit the ability of relayers to use smart contract accounts for relayer, which might not be an issue right now, but could be in the future. Any documentation on why this is not done using _relayer.call ?

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

No branches or pull requests

1 participant