Page MenuHomePhabricator
Differential D12232

[lib] Pass userStateBeforeRecovery to setNewSession call in resolveKeyserverSessionInvalidation
ClosedPublic
Actions

Authored by ashoat on May 28 2024, 10:17 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 14, 10:49 AM
Unknown Object (File)
Sun, Jun 9, 9:50 PM
Unknown Object (File)
Sun, Jun 9, 5:45 PM
Unknown Object (File)
Sun, Jun 9, 11:14 AM
Unknown Object (File)
Sat, Jun 8, 4:30 PM
Unknown Object (File)
Sat, Jun 8, 1:46 PM
Unknown Object (File)
Sat, Jun 8, 11:20 AM
Unknown Object (File)
Sat, Jun 8, 9:40 AM
View All 15 Files
Subscribers
None

Details

Reviewers
inka
tomek
Commits
rCOMMa2a2d84a9b90: [lib] Pass userStateBeforeRecovery to setNewSession call in…
Summary

This resolves ENG-8244. This was an oversight... we should be passing userStateBeforeRecovery to setNewSession here in order to avoid triggering invalidSessionRecovery.

Test Plan
  1. Log in a test user in a multi-keyserver environment
  2. Delete the user's cookie in MariaDB
  3. Background and foreground the app
  4. Confirm that we no longer trigger a loop of the error mentioned in the Linear task

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
lib/
keyserver-conn/
4 lines

Diff 40756

View Options

lib/keyserver-conn/recovery-utils.js

Loading...