Page MenuHomePhabricator

[lib] Don't invalidate a session if the recovery is cancelled
ClosedPublic

Authored by ashoat on Mar 31 2024, 7:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 12, 2:39 PM
Unknown Object (File)
Tue, Nov 12, 2:29 PM
Unknown Object (File)
Tue, Nov 12, 2:29 PM
Unknown Object (File)
Tue, Nov 12, 10:52 AM
Unknown Object (File)
Fri, Nov 1, 10:35 PM
Unknown Object (File)
Oct 11 2024, 10:12 AM
Unknown Object (File)
Oct 11 2024, 10:12 AM
Unknown Object (File)
Oct 11 2024, 9:53 AM
Subscribers
None

Details

Summary

Discovered this issue while testing keyserverAuth-based recovery. We should wait until a non-cancelled recovery fails in order to invalidate the session.

I suspect I hadn't encountered it in prior testing because before introducing keyserverAuth to the dep list, the recovery was unlikely to get cancelled before it ran its course.

Test Plan

Confirm that a cancelled keyserver session recovery doesn't invalidate the session

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable