Page MenuHomePhabricator
Differential D13574

[native] undo changes that removed one time keys from identity v1 login calls
ClosedPublic
Actions

Authored by varun on Oct 2 2024, 8:54 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 2, 6:32 PM
Unknown Object (File)
Sat, Nov 2, 6:32 PM
Unknown Object (File)
Sat, Nov 2, 6:32 PM
Unknown Object (File)
Sat, Nov 2, 6:31 PM
Unknown Object (File)
Sat, Nov 2, 6:27 PM
Unknown Object (File)
Tue, Oct 29, 9:51 PM
Unknown Object (File)
Sun, Oct 27, 6:52 PM
Unknown Object (File)
Wed, Oct 23, 12:07 PM
View All 39 Files
Subscribers
ashoat
tomek

Details

Reviewers
bartek
will
Commits
rCOMMf865d5d62a59: [native] undo changes that removed one time keys from identity v1 login calls
Summary

we determined we actually want to send OTKs here because the device connecting to identity should have a fresh device ID, and therefore 0 pre-existing OTKs in DynamoDB.

https://linear.app/comm/issue/ENG-9440/clients-should-include-otks-on-first-login

Test Plan

log in to existing account from fresh simulator device, confirm that OTKs are present on DynamoDB

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
native/
cpp/
CommonCpp/
NativeModules/
8 lines
16 lines
_generated/
identity-service/
32 lines
native_rust_library/
src/
identity/
12 lines
4 lines
schema/
4 lines

Diff 44943

View Options

native/cpp/CommonCpp/NativeModules/CommRustModule.h

Loading...
View Options

native/cpp/CommonCpp/NativeModules/CommRustModule.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/rustJSI-generated.cpp

Loading...
View Options

native/cpp/CommonCpp/_generated/rustJSI.h

Loading...
View Options

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

Loading...
View Options

native/native_rust_library/src/identity/login.rs

Loading...
View Options

native/native_rust_library/src/lib.rs

Loading...
View Options

native/schema/CommRustModuleSchema.js

Loading...