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

Properly handle failing clear state programs #20

Merged

jasonpaulos merged 3 commits into main from clear-state-error-handling

Jan 4, 2024

Contributor

jasonpaulos commented Jan 4, 2024

Use the new fields introduced in algorand/go-algorand#5842 to properly handle failing clear state programs.

This introduces two behavior changes:

The debugger will show an error if a clear state program rejects or errors. This error does not cause the transaction to fail, and it can be stepped over by the debugger.
Clear state programs will have their app state changes rolled back if they fail. However, the debugger had no way of knowing if this happened prior to this PR. Now it will properly roll back state.

jasonpaulos added 3 commits

January 4, 2024 16:08


          Properly handle clear state program errors

44a4b75


          Changelog update

de06c53


          Format

16c087f

jasonpaulos merged commit b0e025c into main

3 checks passed

jasonpaulos deleted the clear-state-error-handling branch

January 4, 2024 21:16

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet