Page MenuHomePhabricator

[lib] Don't compare identity/keyserver login when not using CSAT
ClosedPublic

Authored by ashoat on May 7 2024, 10:58 AM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 11, 2:11 AM
Unknown Object (File)
Mon, Nov 11, 1:07 AM
Unknown Object (File)
Sun, Nov 10, 10:48 PM
Unknown Object (File)
Sun, Nov 10, 4:54 PM
Unknown Object (File)
Fri, Nov 8, 2:09 PM
Unknown Object (File)
Fri, Nov 8, 10:38 AM
Unknown Object (File)
Fri, Nov 8, 7:32 AM
Unknown Object (File)
Thu, Nov 7, 8:08 PM
Subscribers
None

Details

Summary

In D11692, I moved some logic from being processed on keyserverAuthActionTypes to setNewSessionActionType. I failed to consider that the former action is only used with usingCommServicesAccessToken, but the latter is used regardless of usingCommServicesAccessToken.

This prevents SET_NEW_SESSION for updating the session for legacy keyserver-based login. The session still gets updated later by the auth action (either legacy login or legacy registration), but it would be better to leave the logic as it was before, to avoid potentially breaking anything.

Test Plan
  • Make sure login and registration in legacy single-keyserver environment still works
  • Try both new and old registration flows
  • Make sure keyserver auth returned a different user info than identity login log does not appear

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable