Differential D10815

[native] validate auth result on identity register
ClosedPublic
Actions

Authored by varun on Jan 25 2024, 8:27 AM.

Details

Reviewers

tomek

Commits

rCOMMdd2795b610d0: [native] validate auth result on identity register

Summary

just adding the same assertWithValidator call that we have in logInPasswordUser and getKeyserverKeys

Depends on D10809

Test Plan

registered a new user from native and got back the auth result, which means the validation succeeded

Diff Detail

Repository

rCOMM Comm

Branch

deleteuser (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

varun created this revision.Jan 25 2024, 8:27 AM

Herald added a subscriber: ashoat. · View Herald TranscriptJan 25 2024, 8:27 AM

Harbormaster completed remote builds in B26158: Diff 36135.Jan 25 2024, 8:46 AM

varun requested review of this revision.Jan 25 2024, 8:46 AM

varun added a child revision: D10816: [web] rename callLogIn -> callLegacyLogIn.Jan 25 2024, 10:54 AM

tomek accepted this revision.Jan 26 2024, 2:01 AM

This revision is now accepted and ready to land.Jan 26 2024, 2:01 AM

varun removed a parent revision: D10809: [keyserver] use ONE_TIME_KEYS_NUMBER instead of 10.Jan 29 2024, 8:20 PM

varun added a parent revision: D10455: [web] client method for login password user.

Closed by commit rCOMMdd2795b610d0: [native] validate auth result on identity register (authored by varun). · Explain WhyFeb 7 2024, 9:57 PM

This revision was automatically updated to reflect the committed changes.

varun added a commit: rCOMMdd2795b610d0: [native] validate auth result on identity register.

Revision Contents
Changeset List

Path

Size

native/

identity-service/

identity-service-context-provider.react.js

7 lines

Diff 36135

View Options

native/identity-service/identity-service-context-provider.react.js

[native] validate auth result on identity registerClosedPublicActions