Page MenuHomePhabricator

[web] Introduce `getSignedIdentityKeysBlobSelector(...)` to generate `getSignedIdentityKeysBlob(...)`
ClosedPublic

Authored by atul on Mar 10 2023, 5:32 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Dec 11, 3:46 AM
Unknown Object (File)
Sat, Dec 7, 11:36 PM
Unknown Object (File)
Sat, Dec 7, 11:36 PM
Unknown Object (File)
Sat, Dec 7, 11:35 PM
Unknown Object (File)
Fri, Dec 6, 6:03 AM
Unknown Object (File)
Nov 10 2024, 7:57 PM
Unknown Object (File)
Nov 8 2024, 7:37 AM
Unknown Object (File)
Nov 8 2024, 3:31 AM
Subscribers
None

Details

Summary

Unlike native, where in D7035 getSignedIdentityKeysBlob(...) makes a JSI call, for web we need to construct signedIdentityKeysBlob from the Redux CryptoStore. We generate getSignedIdentityKeysBlob(...) from the new getSignedIdentityKeysBlobSelector and pass it to webGetClientResponsesSelector where it will be passed to the function generated by getClientResponsesSelector (it's a lot of creating a function within a thing that creates a function within a thing that creates a function...)

Now that we have this selector, we can conveniently reuse it in TraditionalLoginForm() and SIWELoginForm(). I'll make that refactor in a subsequent diff.


Depends on D7035

Test Plan

See if ClientClientResponse for serverRequestTypes.SIGNED_IDENTITY_KEYS_BLOB is correctly constructed and received by keyserver:

82f94e.png (1×3 px, 742 KB)

Will also be tested in the diff where we consume getSignedIdentityKeysBlobSelector in TraditionalLoginForm() and SIWELoginForm().

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul published this revision for review.Mar 10 2023, 5:33 PM
This revision is now accepted and ready to land.Mar 11 2023, 5:32 AM

See if ClientClientResponse for serverRequestTypes.SIGNED_IDENTITY_KEYS_BLOB is correctly constructed and received by keyserver.

82f94e.png (1×3 px, 742 KB)

This revision was landed with ongoing or failed builds.Mar 11 2023, 3:49 PM
This revision was automatically updated to reflect the committed changes.