Page MenuHomePhabricator
Differential D9930

[identity] Move UpdateUserPassword RPCs
ClosedPublic
Actions

Authored by bartek on Nov 20 2023, 4:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 15, 6:48 PM
Unknown Object (File)
Sun, Dec 15, 6:48 PM
Unknown Object (File)
Sun, Dec 15, 6:48 PM
Unknown Object (File)
Sun, Dec 15, 6:42 PM
Unknown Object (File)
Wed, Dec 4, 11:56 PM
Unknown Object (File)
Fri, Nov 29, 4:47 PM
Unknown Object (File)
Tue, Nov 26, 5:24 AM
Unknown Object (File)
Nov 22 2024, 2:33 PM
View All 78 Files
Subscribers
ashoat
tomek

Details

Reviewers
varun
Commits
rCOMM717a10d3297b: [identity] Move UpdateUserPassword RPCs
Summary

Moved UpdateUserPassword RPC to the authenticated proto.

Depends on D9929

Test Plan

Registered with Identity Service, was able to log in. Hardcoded call to update_user_password() in native client and verified that password was updated by performning login again.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

bartek created this revision.Nov 20 2023, 4:20 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptNov 20 2023, 4:20 AM
bartek added a child revision: D9931: [identity] Move Logout and DeleteUser RPCs.Nov 20 2023, 4:23 AM
bartek published this revision for review.Nov 20 2023, 4:51 AM
Harbormaster completed remote builds in B24270: Diff 33408.Nov 20 2023, 5:01 AM
varun accepted this revision.Nov 20 2023, 11:38 AM
This revision is now accepted and ready to land.Nov 20 2023, 11:38 AM
varun requested changes to this revision.Nov 27 2023, 10:25 AM

i forgot that we need to update the gRPC web codegen files. can you please do that?

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

i forgot that we need to update the gRPC web codegen files. can you please do that?

I created D10064 to update them all at once

varun accepted this revision.Nov 29 2023, 6:09 AM
This revision is now accepted and ready to land.Nov 29 2023, 6:09 AM
Closed by commit rCOMM717a10d3297b: [identity] Move UpdateUserPassword 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: rCOMM717a10d3297b: [identity] Move UpdateUserPassword RPCs.
bartek mentioned this in D10573: Update gRPC-web generated files.Jan 8 2024, 2:49 AM

Revision Contents
Changeset List

PathSize
native/
native_rust_library/
src/
16 lines
services/
identity/
src/
80 lines
grpc_services/
74 lines
4 lines
shared/
protos/
30 lines
35 lines

Diff 34173

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

services/identity/src/main.rs

Loading...
View Options

shared/protos/identity_authenticated.proto

Loading...
View Options

shared/protos/identity_client.proto

Loading...