Page MenuHomePhabricator

[identity] delete user RPC
ClosedPublic

Authored by varun on Apr 19 2023, 2:13 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 12, 7:21 PM
Unknown Object (File)
Tue, Mar 12, 7:15 PM
Unknown Object (File)
Thu, Mar 7, 7:03 AM
Unknown Object (File)
Thu, Mar 7, 5:56 AM
Unknown Object (File)
Thu, Mar 7, 5:22 AM
Unknown Object (File)
Thu, Mar 7, 5:21 AM
Unknown Object (File)
Thu, Mar 7, 12:26 AM
Unknown Object (File)
Feb 21 2024, 9:32 PM
Subscribers

Details

Summary

Depends on D7541

Test Plan

registered a new user, then successfully deleted it

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

Harbormaster returned this revision to the author for changes because remote builds failed.Apr 19 2023, 2:39 PM
Harbormaster failed remote builds in B18558: Diff 25423!
shared/protos/identity_client.proto
222–225

Can you clarify why these are being added? Is it impossible to look up an accessToken just by itself, and we always need to include userID and deviceIDKey in order to validate it? (Guessing this is the case based on VerifyUserAccessTokenRequest)

shared/protos/identity_client.proto
222–225

yeah in our access token table, the primary key is user ID + device ID (ed25519 signing key)

shared/protos/identity_client.proto
222–225

Thanks!

This revision is now accepted and ready to land.Apr 24 2023, 1:08 PM
This revision was automatically updated to reflect the committed changes.