Skip to content

Conversation

@gab8i
Copy link
Contributor

@gab8i gab8i commented Dec 4, 2025

When beginning a session, ensure that the oldest overlay’s previous root matches the current state root. This prevents the creation of invalid sessions that could reuse a chain of overlays that were previously committed or or a chain that is no longer valid because another overlay chain has been committed.

@gab8i
Copy link
Contributor Author

gab8i commented Dec 4, 2025

@rphmeier @pepyakin what do you think about having a begin_session method that starts returning a Result? It’s somewhat of an api change, but it seems necessary to me. Do you have any other approach in mind?

@gab8i gab8i force-pushed the gm_ensure_session_validity branch from ce57dba to f4eb962 Compare December 4, 2025 16:25
@gab8i gab8i force-pushed the gm_overlay_deletions_fix branch from 9151587 to c8b3c95 Compare December 4, 2025 16:37
gabriele-0201 added 2 commits December 4, 2025 17:37
When beginning a session, ensure that the oldest overlay’s
previous root matches the current state root. This prevents the
creation of invalid sessions that could reuse a chain of overlays
that were previously committed or or a chain that is no longer
valid because another overlay chain has been committed.
@gab8i gab8i force-pushed the gm_ensure_session_validity branch from f4eb962 to 11d1764 Compare December 4, 2025 16:39
Base automatically changed from gm_overlay_deletions_fix to master December 5, 2025 16:47
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