fix(contract): extend Soroban instance storage TTL to prevent fund lo… - #42
Merged
thisisouvik merged 1 commit intoAug 31, 2026
Conversation
…ckup The AutopilotVault never extended its instance storage TTL. Soroban instance entries expire after a fixed number of ledgers, so a vault that sat untouched past MAX_ENTRY_LIFETIME would lose its Owner/Engine/ IsInitialized entries and panic on withdraw(), locking funds with no recovery path. Bump the instance TTL to ~30 days whenever it drops below ~1 day, from initialize() and from every state-touching entry point (withdraw, engine_execute). Also add a permissionless extend_ttl() keep-alive so the vault can be kept reachable without moving funds. Fixes thisisouvik#7 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
@Soumen1080 is attempting to deploy a commit to the thisisouvik's projects Team on Vercel. A member of the Team first needs to authorize it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
close #7
Committed on the branch. The remaining steps — pushing and opening/merging a PR — need GitHub auth, and gh auth status reports you're not logged in, so I can't do those from here.
Done:
Branch fix/contract-storage-ttl-extension, commit bc92dd2
lib.rs: TTL bump helper (~30 days, threshold ~1 day) called from initialize, withdraw, engine_execute, plus a permissionless extend_ttl() keep-alive
test.rs: 2 new tests; all 6 pass, and the TTL assertion verifiably fails if the bump is removed