Skip to content

Conversation

@anton-lisanin
Copy link
Collaborator

Remove the account blocking mechanism and introduce the account revision.
Each time the account is changed, increment the revision by 1.

At the beginning of the first iteration, store an entire transaction data and revisions for all accounts in the Holder. In each iteration, compare the account revision with the one stored in the Holder. If they do not match, reset the EVM state to the first iteration.

@anton-lisanin anton-lisanin force-pushed the ndev_2522_blocked_accounts branch 2 times, most recently from 6e58892 to 331e1ca Compare February 19, 2024 11:28
afalaleev
afalaleev previously approved these changes Feb 19, 2024
@anton-lisanin anton-lisanin added the fullTestSuite Run OZ tests and part of dapps tests label Feb 20, 2024
@anton-lisanin anton-lisanin force-pushed the ndev_2522_blocked_accounts branch from 7721896 to ed0f24d Compare February 20, 2024 12:59
@anton-lisanin anton-lisanin force-pushed the ndev_2522_blocked_accounts branch from ed0f24d to dc07639 Compare February 20, 2024 13:06
@neonlabstech
Copy link

Dapps report

@anton-lisanin anton-lisanin merged commit 3a179a9 into develop Feb 22, 2024
@anton-lisanin anton-lisanin deleted the ndev_2522_blocked_accounts branch February 22, 2024 07:44
anton-lisanin added a commit that referenced this pull request Feb 26, 2024
* Add Account Revision (#276)
* NDEV-2651 Remove syscall_stabs (#279)
* NDEV-2620: Add executor_state: &impl Database parameter to EventListener::event method (#273)
* add keys for Miro, DeBank/ Rabby and Alfie for devnet WL
---------

Co-authored-by: Semen Medvedev <[email protected]>
Co-authored-by: Miroslav Nedelchev <[email protected]>
Co-authored-by: Andrei Silviu Dragnea <[email protected]>
Co-authored-by: s-medvedev <[email protected]>
Co-authored-by: Kristina Nikolaeva <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fullTestSuite Run OZ tests and part of dapps tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants