Page MenuHomePhabricator

[lib] Discard keyserver usernames for keyserverAuthActionTypes
ClosedPublic

Authored by inka on Jun 20 2024, 4:16 AM.
Tags
None
Referenced Files
F3495719: D12509.id41687.diff
Thu, Dec 19, 9:44 AM
F3495718: D12509.id41596.diff
Thu, Dec 19, 9:44 AM
F3495717: D12509.id41550.diff
Thu, Dec 19, 9:44 AM
F3495708: D12509.id.diff
Thu, Dec 19, 9:44 AM
F3495705: D12509.diff
Thu, Dec 19, 9:44 AM
Unknown Object (File)
Tue, Dec 17, 3:31 PM
Unknown Object (File)
Sat, Dec 14, 11:03 PM
Unknown Object (File)
Mon, Dec 9, 9:21 AM
Subscribers

Details

Summary

issue: ENG-8365
update_relationships throws for viewers own id, because it doesn't make sense to be updating users relationship with themselves. But the user should always have their own user info sent during keyserver auth, so there is no need to fetch it, for any action

Test Plan

Logged in. Checked that usernames for both the current user and other users are null after keyserver auth and get fetched from identity. Checked that the invalid_user error doesn't show up.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Jun 20 2024, 4:32 AM
lib/handlers/user-infos-handler.react.js
71–73 ↗(On Diff #41550)

I should check again if userIDsWithoutOwnID is not empty

This revision is now accepted and ready to land.Jun 21 2024, 3:24 AM