-
Notifications
You must be signed in to change notification settings - Fork 15
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
whitelisted senders have unlimited allowance and proper checks for ft_transfer to burn address #41
Conversation
anthony-near
commented
Oct 31, 2023
•
edited
Loading
edited
- whitelisted senders have unlimited allowance on startup for fastauth. Fixes: https://github.com/near/pagoda-relayer-rs-fastauth/issues/17 by setting the initial allowance for all whitelisted senders to u64::max.
- proper check for ft_transfer to burn address for pay with FT
- rustfmt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's hard to predict if it works without integration tests TBH.
The fact that we are mixing concepts of senders and receivers is questionable.
@anthony-near once the docker image is updated, we can test this functionality by rerunning tests here: near/mpc#347 |
I know it's not as good as an integration test, but I ran it locally and called the |