Page MenuHomePhabricator

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

Authored by ashoat on Tue, May 7, 10:58 AM.
Tags
None
Referenced Files
F1786224: D11927.id39938.diff
Sat, May 18, 4:16 PM
Unknown Object (File)
Fri, May 17, 10:17 PM
Unknown Object (File)
Fri, May 17, 9:13 AM
Unknown Object (File)
Sun, May 12, 10:21 AM
Unknown Object (File)
Sat, May 11, 7:19 PM
Unknown Object (File)
Sat, May 11, 4:22 PM
Unknown Object (File)
Fri, May 10, 1:00 PM
Unknown Object (File)
Fri, May 10, 4:23 AM
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