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

AVM: Share resource arrays across transactions #5035

Merged
merged 30 commits into from
Apr 7, 2023
Merged

AVM: Share resource arrays across transactions #5035

merged 30 commits into from
Apr 7, 2023

Commits on Mar 20, 2023

  1. Initial implementation. 

    Needs unit tests for access to resources in non-appl calls.
    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    5051793 View commit details
    Browse the repository at this point in the history

  2. Re-org availability checks in prep for changes

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    97e2269 View commit details
    Browse the repository at this point in the history

  3. Checkpoint with new approach, needs v9 -> v8 tests 

    Does not yet have the extra check to ensure v9 can't expose new
resources by putting accounts from one tx and assets from another into
static arrays of a called v8 app.
    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    bbaa278 View commit details
    Browse the repository at this point in the history

  4. Adjust names after rebase

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    857d17e View commit details
    Browse the repository at this point in the history

  5. Add checks for inner txn resource access 

    Also, enhance EvalDelta to allow modifications of local states for
addresses that are not in txn.Accounts.  This probably needs a bit of
work on the REST endpoints that emit them.
    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    bc2a5af View commit details
    Browse the repository at this point in the history

  6. Account for "shared" indexes in EvalDelta

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    726be12 View commit details
    Browse the repository at this point in the history

  7. oas2 fixups

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    a48886e View commit details
    Browse the repository at this point in the history

  8. e2e test for shared resource access

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    28aa602 View commit details
    Browse the repository at this point in the history

  9. Several more tests, and handling created apps/accounts

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    86dfb59 View commit details
    Browse the repository at this point in the history

  10. allocbound

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    e9dfd02 View commit details
    Browse the repository at this point in the history

  11. coverage

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    bea0c92 View commit details
    Browse the repository at this point in the history

  12. Test account passing across versions for more coverage

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    ef20192 View commit details
    Browse the repository at this point in the history

  13. Bob CR

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    8b09e82 View commit details
    Browse the repository at this point in the history

  14. Fix suggested by @jasonpaulos 

    See TestAccessMyLocals for evidence of proper beahior
    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    c70f0b6 View commit details
    Browse the repository at this point in the history

  15. Add (and use) a simpler way to do cross version app testing

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    b106666 View commit details
    Browse the repository at this point in the history

  16. Fix app_opted_in for resource sharing, and initial spec changes

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    d60380c View commit details
    Browse the repository at this point in the history

  17. Improvements from CR. Some more tests to come.

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    bf98ae4 View commit details
    Browse the repository at this point in the history

  18. New tests, including v9 calling v8 showing blocked access

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    cf2bfeb View commit details
    Browse the repository at this point in the history

  19. Zeph CR typos

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    2bd96e1 View commit details
    Browse the repository at this point in the history

  20. regenetate

    @jannotti
    jannotti committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    1ef746b View commit details
    Browse the repository at this point in the history

Commits on Mar 22, 2023

  1. Comment clarity

    @jannotti
    jannotti committed Mar 22, 2023
    Configuration menu
    Copy the full SHA
    8f738fb View commit details
    Browse the repository at this point in the history

Commits on Mar 23, 2023

  1. Pavel CR

    @jannotti
    jannotti committed Mar 23, 2023
    Configuration menu
    Copy the full SHA
    41f4f86 View commit details
    Browse the repository at this point in the history

Commits on Mar 30, 2023

  1. Add tests for interpretation of integers as resource or slot

    @jannotti
    jannotti committed Mar 30, 2023
    Configuration menu
    Copy the full SHA
    56b2b55 View commit details
    Browse the repository at this point in the history

  2. Remove use of slots for app and asset indexes 

    This changes the behavior of several opcodes in v9.  We will probably
rename them in v9, to force writers to notice, but that is not in this
commit.
    @jannotti
    jannotti committed Mar 30, 2023
    Configuration menu
    Copy the full SHA
    b4e63e8 View commit details
    Browse the repository at this point in the history

Commits on Mar 31, 2023

  1. Remove slot usage for account access opcodes

    @jannotti
    jannotti committed Mar 31, 2023
    Configuration menu
    Copy the full SHA
    946309e View commit details
    Browse the repository at this point in the history

Commits on Apr 2, 2023

  1. adjust ledger tests for v9

    @jannotti
    jannotti committed Apr 2, 2023
    Configuration menu
    Copy the full SHA
    ccdd661 View commit details
    Browse the repository at this point in the history

Commits on Apr 3, 2023

  1. add extra test discussed with Pavel

    @jannotti
    jannotti committed Apr 3, 2023
    Configuration menu
    Copy the full SHA
    2239746 View commit details
    Browse the repository at this point in the history

Commits on Apr 4, 2023

  1. Merge remote-tracking branch 'upstream/master' into avm-group-resources

    @jannotti
    jannotti committed Apr 4, 2023
    Configuration menu
    Copy the full SHA
    3ad609f View commit details
    Browse the repository at this point in the history

  2. fix long standing typo

    @jannotti
    jannotti committed Apr 4, 2023
    Configuration menu
    Copy the full SHA
    0eb3923 View commit details
    Browse the repository at this point in the history

Commits on Apr 7, 2023

  1. Final (?) cr fixes and an obscure bug fix 

    I already have tests for the bug fix (the change is in
EvalContext.availableAccount(), but they rely on a fair bit of code I
have in the next PR, so I will include it there.
    @jannotti
    jannotti committed Apr 7, 2023
    Configuration menu
    Copy the full SHA
    530c76a View commit details
    Browse the repository at this point in the history