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)
Thu, Apr 24, 2:26 PM
Unknown Object (File)
Thu, Apr 17, 5:10 AM
Unknown Object (File)
Wed, Apr 16, 10:33 PM
Unknown Object (File)
Wed, Apr 16, 9:15 PM
Unknown Object (File)
Tue, Apr 15, 11:08 PM
Unknown Object (File)
Thu, Apr 10, 12:59 AM
Unknown Object (File)
Mon, Apr 7, 6:03 AM
Unknown Object (File)
Mon, Mar 31, 10:02 PM
View All 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
No Lint Coverage
Unit
No Test Coverage

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 33408

View Options

native/native_rust_library/src/lib.rs

Show All 21 Lines declare export class Notification {
topic: string; topic: string;
id: string; id: string;
expiry: number; expiry: number;
priority: number; priority: number;
collapseId: string; collapseId: string;
pushType: NotificationPushType; pushType: NotificationPushType;
threadId: string; threadId: string;
payload: any; payload: any;
badge: number; badge: ?number;
sound: string; sound: string;
contentAvailable: boolean; contentAvailable: boolean;
mutableContent: boolean; mutableContent: boolean;
urlArgs: string[]; urlArgs: string[];
// Detailed explanation of this field can be found here:
// https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/generating_a_remote_notification?language=objc#2943363
// More fields can be added here, if they ever need to
// be accessed from apn.Notification instance
aps: {
+badge: string | number,
+alert: string,
+'thread-id': string,
+'mutable-content': boolean,
+sound: string,
...
};
} }
declare type ProviderToken = {| declare type ProviderToken = {|
+key: string, +key: string,
+keyId: string, +keyId: string,
+teamId: string, +teamId: string,
|}; |};
declare export type ProviderOptions = { declare export type ProviderOptions = {
▲ Show 20 Lines • Show All 398 Lines • Show Last 20 Lines
View Options

services/identity/src/client_service.rs

Show All 21 Lines declare export class Notification {
topic: string; topic: string;
id: string; id: string;
expiry: number; expiry: number;
priority: number; priority: number;
collapseId: string; collapseId: string;
pushType: NotificationPushType; pushType: NotificationPushType;
threadId: string; threadId: string;
payload: any; payload: any;
badge: number; badge: ?number;
sound: string; sound: string;
contentAvailable: boolean; contentAvailable: boolean;
mutableContent: boolean; mutableContent: boolean;
urlArgs: string[]; urlArgs: string[];
// Detailed explanation of this field can be found here:
// https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/generating_a_remote_notification?language=objc#2943363
// More fields can be added here, if they ever need to
// be accessed from apn.Notification instance
aps: {
+badge: string | number,
+alert: string,
+'thread-id': string,
+'mutable-content': boolean,
+sound: string,
...
};
} }
declare type ProviderToken = {| declare type ProviderToken = {|
+key: string, +key: string,
+keyId: string, +keyId: string,
+teamId: string, +teamId: string,
|}; |};
declare export type ProviderOptions = { declare export type ProviderOptions = {
▲ Show 20 Lines • Show All 398 Lines • Show Last 20 Lines
View Options

services/identity/src/grpc_services/authenticated.rs

Show All 21 Lines declare export class Notification {
topic: string; topic: string;
id: string; id: string;
expiry: number; expiry: number;
priority: number; priority: number;
collapseId: string; collapseId: string;
pushType: NotificationPushType; pushType: NotificationPushType;
threadId: string; threadId: string;
payload: any; payload: any;
badge: number; badge: ?number;
sound: string; sound: string;
contentAvailable: boolean; contentAvailable: boolean;
mutableContent: boolean; mutableContent: boolean;
urlArgs: string[]; urlArgs: string[];
// Detailed explanation of this field can be found here:
// https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/generating_a_remote_notification?language=objc#2943363
// More fields can be added here, if they ever need to
// be accessed from apn.Notification instance
aps: {
+badge: string | number,
+alert: string,
+'thread-id': string,
+'mutable-content': boolean,
+sound: string,
...
};
} }
declare type ProviderToken = {| declare type ProviderToken = {|
+key: string, +key: string,
+keyId: string, +keyId: string,
+teamId: string, +teamId: string,
|}; |};
declare export type ProviderOptions = { declare export type ProviderOptions = {
▲ Show 20 Lines • Show All 398 Lines • Show Last 20 Lines
View Options

services/identity/src/main.rs

Show All 21 Lines declare export class Notification {
topic: string; topic: string;
id: string; id: string;
expiry: number; expiry: number;
priority: number; priority: number;
collapseId: string; collapseId: string;
pushType: NotificationPushType; pushType: NotificationPushType;
threadId: string; threadId: string;
payload: any; payload: any;
badge: number; badge: ?number;
sound: string; sound: string;
contentAvailable: boolean; contentAvailable: boolean;
mutableContent: boolean; mutableContent: boolean;
urlArgs: string[]; urlArgs: string[];
// Detailed explanation of this field can be found here:
// https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/generating_a_remote_notification?language=objc#2943363
// More fields can be added here, if they ever need to
// be accessed from apn.Notification instance
aps: {
+badge: string | number,
+alert: string,
+'thread-id': string,
+'mutable-content': boolean,
+sound: string,
...
};
} }
declare type ProviderToken = {| declare type ProviderToken = {|
+key: string, +key: string,
+keyId: string, +keyId: string,
+teamId: string, +teamId: string,
|}; |};
declare export type ProviderOptions = { declare export type ProviderOptions = {
▲ Show 20 Lines • Show All 398 Lines • Show Last 20 Lines
View Options

shared/protos/identity_authenticated.proto

Show All 21 Lines declare export class Notification {
topic: string; topic: string;
id: string; id: string;
expiry: number; expiry: number;
priority: number; priority: number;
collapseId: string; collapseId: string;
pushType: NotificationPushType; pushType: NotificationPushType;
threadId: string; threadId: string;
payload: any; payload: any;
badge: number; badge: ?number;
sound: string; sound: string;
contentAvailable: boolean; contentAvailable: boolean;
mutableContent: boolean; mutableContent: boolean;
urlArgs: string[]; urlArgs: string[];
// Detailed explanation of this field can be found here:
// https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/generating_a_remote_notification?language=objc#2943363
// More fields can be added here, if they ever need to
// be accessed from apn.Notification instance
aps: {
+badge: string | number,
+alert: string,
+'thread-id': string,
+'mutable-content': boolean,
+sound: string,
...
};
} }
declare type ProviderToken = {| declare type ProviderToken = {|
+key: string, +key: string,
+keyId: string, +keyId: string,
+teamId: string, +teamId: string,
|}; |};
declare export type ProviderOptions = { declare export type ProviderOptions = {
▲ Show 20 Lines • Show All 398 Lines • Show Last 20 Lines
View Options

shared/protos/identity_client.proto

Loading...