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 transaction size #3274

Closed
bowenwang1996 opened this issue Sep 2, 2020 · 0 comments · Fixed by #4107
Closed

Limit transaction size #3274

bowenwang1996 opened this issue Sep 2, 2020 · 0 comments · Fixed by #4107
Assignees
@Longarithm
Labels
A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc)

Comments

@bowenwang1996
Copy link
Collaborator

Currently we have some limits on how large a valid transaction can be, but it is too relaxed. For example, a transaction that contains 100 actions each of which deploys a 4MB contract can make the transaction unnecessarily large whereas in reality it will definitely exceed gas limit. We should introduce a more reasonable limit to the size of transactions to roughly match the gas limit.
@bowenwang1996 bowenwang1996 added the A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc) label Sep 2, 2020
@bowenwang1996 bowenwang1996 mentioned this issue Oct 6, 2020
Open
@Longarithm Longarithm linked a pull request Mar 15, 2021 that will close this issue
Set transaction size limit #4107
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Longarithm Longarithm

Labels
A-transaction-runtime Area: transaction runtime (transaction and receipts processing, state transition, etc)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Set transaction size limit near/nearcore
2 participants
@Longarithm @bowenwang1996