Page MenuHomePhabricator
Differential D9931

[identity] Move Logout and DeleteUser RPCs
ClosedPublic
Actions

Authored by bartek on Nov 20 2023, 4:23 AM.
Tags
None
Referenced Files
F3348029: D9931.diff
Fri, Nov 22, 1:40 PM
Unknown Object (File)
Sun, Nov 10, 10:30 AM
Unknown Object (File)
Sun, Nov 10, 10:25 AM
Unknown Object (File)
Sat, Nov 9, 3:48 PM
Unknown Object (File)
Sat, Nov 9, 8:20 AM
Unknown Object (File)
Oct 20 2024, 4:49 PM
Unknown Object (File)
Oct 18 2024, 6:16 AM
Unknown Object (File)
Oct 18 2024, 6:16 AM
View All 89 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
Commits
rCOMM04526b2637a0: [identity] Move Logout and DeleteUser RPCs
Summary

Moved Logout and DeleteUser RPCs to the authenticated proto. Updated client calls accordingly.

Depends on D9930

Test Plan

Verified that logout is callable via auth client. Verified that DeleteUser deletes item from users table (called this in native commRustModule to verify client working)

Diff Detail

Repository
rCOMM Comm
Branch
barthap/identity-move-rpc
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Nov 20 2023, 4:23 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 20 2023, 4:23 AM
Harbormaster failed remote builds in B24271: Diff 33409!Nov 20 2023, 4:49 AM
bartek updated this revision to Diff 33413.Nov 20 2023, 4:54 AM

Fix import

Harbormaster completed remote builds in B24275: Diff 33413.Nov 20 2023, 6:04 AM
bartek published this revision for review.Nov 23 2023, 1:48 AM
bartek added a child revision: D9957: [identity] Move X3DH RPCs to auth proto.Nov 23 2023, 1:53 AM
varun requested changes to this revision.Nov 27 2023, 10:24 AM

can you please also update the gRPC web codegen files?

This revision now requires changes to proceed.Nov 27 2023, 10:24 AM
bartek requested review of this revision.Nov 28 2023, 8:04 AM
In D9931#292402, @varun wrote:

can you please also update the gRPC web codegen files?

I updated these in D10064

bartek mentioned this in D10064: [web] Update gRPC-web generated files.Nov 29 2023, 12:08 AM
varun accepted this revision.Nov 29 2023, 6:10 AM
This revision is now accepted and ready to land.Nov 29 2023, 6:10 AM
Closed by commit rCOMM04526b2637a0: [identity] Move Logout and DeleteUser RPCs (authored by bartek). · Explain WhyDec 4 2023, 2:43 AM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMM04526b2637a0: [identity] Move Logout and DeleteUser RPCs.

Revision Contents
Changeset List

PathSize
native/
native_rust_library/
src/
21 lines
services/
identity/
src/
91 lines
grpc_services/
38 lines
shared/
protos/
5 lines
24 lines

Diff 33413

View Options

native/native_rust_library/src/lib.rs

Loading...
View Options

services/identity/src/client_service.rs

Loading...
View Options

services/identity/src/grpc_services/authenticated.rs

Loading...
View Options

shared/protos/identity_authenticated.proto

Loading...
View Options

shared/protos/identity_client.proto

Loading...