Skip to content
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

in an atomic transaction of malleable tokens, should mint first and transfer later, or do that in pairs. #8

Open
SmartLayer opened this issue Mar 6, 2024 · 2 comments

Comments

@SmartLayer
Copy link
Contributor

https://sepolia.etherscan.io/tx/0xf0223806820911beddaa42f2a055818f40b4c8162b88284966ceba1d9d10dde1

shows that address transferred 2727 tokens to 0x2F21dC12dd43bd15b86643332041ab97010357D7

however, the transaction records shows burn first:
image

Users who didn't scroll to the bottom might sweat a lot before learing what's really going on
image

@SmartLayer SmartLayer changed the title Let's transfer first, burn later. in an atomic transaction of malleable tokens, should mint first and transfer later, or do that in pairs. Mar 6, 2024
@oleggrib
Copy link
Collaborator

oleggrib commented Mar 6, 2024

it can use more gas if we decide to make pairs,
but I can try to do it to see gas usage
tell me if you need me to do it in pairs @SmartLayer

@oleggrib
Copy link
Collaborator

oleggrib commented Mar 8, 2024

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

No branches or pull requests

2 participants