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

Meter (and charge for) the transaction storage check #3834

Closed
janezpodhostnik opened this issue Jan 19, 2023 · 2 comments · Fixed by #5497
Closed

Meter (and charge for) the transaction storage check #3834

janezpodhostnik opened this issue Jan 19, 2023 · 2 comments · Fixed by #5497
Assignees
Labels
Execution Cadence Execution Team Improvement

Comments

@janezpodhostnik
Copy link
Contributor

janezpodhostnik commented Jan 19, 2023

Problem Definition

Storage Check1 is un-metered. This is a problem, because users can control how long (complex) the storage check is.

If a transaction touches the registers of more accounts, then the storage check will be longer.

Proposed Solution

Meter the storage check as a normal part of the transaction.

The technical change is very simple, but this would mean that the computation used of transactions would change, so needs to be announced well in advance and we might want to bundle it with any other fee changes.

Release notes

Please make sure to mention this in the release notes. This will make all transactions a tiny bit more expensive, but transactions that touch a lot of accounts might be significantly more expensive.

This is change is needed to close a gap in metering.

Definition of Done

  • [] The storage check is metered

Footnotes

  1. The part of the transaction, where all account that had any of their storage registers changed, are checked if their storage capacity is more then their storage used. If not the transaction fails.

@janezpodhostnik
Copy link
Contributor Author

Could be released together with #5026

@j1010001
Copy link
Member

waiting for a change to fees so we can deploy it together (could be in HCU)

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

Successfully merging a pull request may close this issue.

2 participants