Skip to content

[Certora] Responsive guardian strategy#464

Open
QGarchery wants to merge 5 commits into
mainfrom
guardian-responsive
Open

[Certora] Responsive guardian strategy#464
QGarchery wants to merge 5 commits into
mainfrom
guardian-responsive

Conversation

@QGarchery

Copy link
Copy Markdown
Contributor

No description provided.

@QGarchery QGarchery self-assigned this Jul 18, 2026
@QGarchery QGarchery mentioned this pull request Jul 18, 2026
28 tasks

@MathisGD MathisGD left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

Comment thread certora/README.md Outdated
Comment thread certora/README.md Outdated
QGarchery and others added 2 commits July 20, 2026 09:01
Co-authored-by: MathisGD <74971347+MathisGD@users.noreply.github.com>
Signed-off-by: Quentin Garchery <garchery.quentin@gmail.com>
// Show that revoking the pending guardian restarts a full timelock before any guardian change.
rule guardianUpdateTimeAfterRevoke(env e) {
// Assume that there is no pending timelock, which the guardian can ensure by revoking it.
require pendingTimelock_().validAt == 0;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a reason why there is a require instead of calling revokePendingTimelock(e)?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not a strong one, but the guardian may not need to call revokePendingTimelock, if no pending timelock have been submitted. Seems a bit more natural, since the guardian won't revoke everything in practice, and may be ok with a reduced notice period

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.

3 participants