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.

Sign up for GitHub

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

Jump to bottom

Limit gas limit for free claims #1482

Closed
arnaubennassar opened this issue Dec 22, 2022 · 2 comments · Fixed by #1551
Closed

Limit gas limit for free claims #1482

arnaubennassar opened this issue Dec 22, 2022 · 2 comments · Fixed by #1551
Assignees
@ARR552
Labels
improvement

Comments

@arnaubennassar
Copy link
Member

arnaubennassar commented Dec 22, 2022 •
edited

value should be 150k
@0-Rainer-Stoff-0
Copy link

value should be 150k

@arnaubennassar
Copy link
Member Author

An alternative approach could be checking if the destination address is a SC, and only allow non SC receivers

@ARR552 ARR552 mentioned this issue Jan 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ARR552 ARR552

Labels
improvement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Gas limit 150K + tests 0xPolygonHermez/zkevm-node
3 participants
@arnaubennassar @ARR552 @0-Rainer-Stoff-0