Page MenuHomePhabricator
Differential D11327

[keyserver] Remove deprectaed uses of updateTypes.UPDATE_CURRENT_USER
ClosedPublic
Actions

Authored by inka on Mar 14 2024, 6:10 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 5:08 AM
Unknown Object (File)
Fri, Nov 8, 5:08 AM
Unknown Object (File)
Fri, Nov 8, 5:02 AM
Unknown Object (File)
Fri, Nov 8, 4:42 AM
Unknown Object (File)
Oct 3 2024, 9:58 AM
Unknown Object (File)
Oct 3 2024, 9:58 AM
Unknown Object (File)
Oct 3 2024, 9:58 AM
Unknown Object (File)
Oct 3 2024, 9:58 AM
View All 46 Files
Subscribers
ashoat
tomek

Details

Reviewers
michal
kamil
ginsu
Commits
rCOMMcbe1a868fd6d: [keyserver] Remove deprectaed uses of updateTypes.UPDATE_CURRENT_USER
Summary

issue: ENG-7205
The two removed files are scripts. They were not used anywhere in the code (they were supposed to be run manually). Now that identity service is the administrator of user data, it doesn't make sense to run scripts that change user data on keyserver.

In accountUpdater updates were created back when password was stored in CurrentUserInfo. See D1470. Now accountUpdater is only used for updating passwords, so there are no changes to curentUserInfo it can make

Test Plan

Tested that it is still possible to change the password and then login on this user account from both web and native

Diff Detail

Repository
rCOMM Comm
Branch
inka/state_sync
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
keyserver/
src/
responders/
4 lines
scripts/
updaters/
16 lines

Diff 38061

View Options

keyserver/src/responders/user-responders.js

Loading...
View Options

keyserver/src/scripts/merge-users.js

Loading...
View Options

keyserver/src/scripts/rename-user.js

Loading...
View Options

keyserver/src/updaters/account-updaters.js

Loading...