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

fix: disallow sending preconf txs with future nonces #13

Conversation

radkomih
Copy link
Collaborator

Disallow pre-confirmation txs with future nonces, since there is no guarantee that they will be valid and included in a block

@radkomih radkomih self-assigned this Aug 27, 2024
@radkomih radkomih force-pushed the refactor/move-txlist-trigger-in-geth branch from 31bb8fb to 4a9d969 Compare September 19, 2024 09:15
@radkomih radkomih merged commit 99c7564 into refactor/move-txlist-trigger-in-geth Sep 24, 2024
@radkomih radkomih deleted the fix/sending-preconf-txs-with-future-nonces branch September 24, 2024 09:09
failfmi pushed a commit that referenced this pull request Oct 16, 2024
* get slots and tx list config from storage

* add MaxProposedTxListsPerEpoch param

* refactor

* revert flags

* fix: non-initialized snapshot

* fix: return errors and refactor

* refactor: calc current slot/epoch and store assigned slots

* refactor

* update L1 genesis timestamp

* wip: refactor

* fixes

* use lock for slot estimates

* remove tx for missed slots

* fixes

* refactor worker locks

* handle other non-preconf txs

* fix: disallow sending preconf txs with future nonces (#13)

* update/reset txslotsnapshot gas/bytes

* fix proposing non-preconf txs

* fix: update geth internal_l2 genesis

* simulate txs per slot and calc gas/bytes

* fix processing of future epoch slots

* rework workers & locks. remove logging goroutine. script for spamming

* fix preconf tx hash

* remove slot offset

* fix 0 slot

* fix block number in preconf receipts

---------

Co-authored-by: Vladimir Trifonov <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants