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

New unrestricted-transfer-from documentation #320

Merged
merged 4 commits into from
Aug 12, 2024

Conversation

tomasavola
Copy link
Contributor

No description provided.

@tomasavola tomasavola added the documentation Improvements or additions to documentation label Aug 9, 2024
@tomasavola tomasavola self-assigned this Aug 9, 2024
Copy link

github-actions bot commented Aug 9, 2024

📘 Documentation Test Summary 📘

✅ The test build of the documentation has completed successfully!
📝 The changes will be published upon merge to the main branch.


An user Alice can approve a contract to spend their tokens. An user Bob can call that contract, use that allowance to send themselves Alice's tokens.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Creo que esta explicación podría estar más clara. Podríamos poner algo así: "The absence of proper authorization checks for sensitive operations, like transfer_form, can lead to the loss of funds or other undesired consequences. For example, if a user, Alice, approves a contract to spend her tokens, and the contract lacks proper authorization checks, another user, Bob, could invoke the contract and potentially transfer Alice's tokens to himself without her explicit consent.

@matiascabello matiascabello merged commit 7f08850 into main Aug 12, 2024
2 checks passed
@matiascabello matiascabello deleted the migrate-unrestricted-transfer-from-documentation branch August 12, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants