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)
Mon, Dec 16, 7:53 PM
Unknown Object (File)
Mon, Dec 16, 8:38 AM
Unknown Object (File)
Mon, Dec 16, 8:38 AM
Unknown Object (File)
Mon, Dec 16, 8:37 AM
Unknown Object (File)
Mon, Dec 16, 7:43 AM
Unknown Object (File)
Fri, Dec 6, 6:59 PM
Unknown Object (File)
Nov 23 2024, 11:59 PM
Unknown Object (File)
Nov 23 2024, 11:50 PM
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