Paths

Table of Contentst

Differential D6124

[native] Generate `primaryIdentityPublicKey` and pass to `landing/SIWE`
ClosedPublic
Actions

Authored by atul on Dec 31 2022, 12:52 AM.

Tags

None

Referenced Files

None

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMM1220a5821da4: [native] Generate `primaryIdentityPublicKey` and pass to `landing/SIWE`

Summary

Make necessary JSI calls to get primaryIdentityPublicKey and set siwe-primary-identity-public-key header before loading landing/SIWE webview.

Depends on D6115

Test Plan

Logged primaryIdentityPublicKey value and ensured it was as expected.

In subsequent diff will capture the value from header and pass to landing/SIWE component so it can be included in the signed SIWEMessage.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Dec 31 2022, 12:52 AM

atul published this revision for review.Dec 31 2022, 12:53 AM

Harbormaster completed remote builds in B14993: Diff 20462.Dec 31 2022, 1:06 AM

atul edited the summary of this revision. (Show Details)Dec 31 2022, 2:02 AM

atul added a parent revision: D6115: [native] Modify `getUserPublicKey()` JSI function to return `ClientPublicKeys`.

atul added a child revision: D6125: [lib] Introduce `isValidPrimaryIdentityPublicKey`.Dec 31 2022, 2:03 AM

ashoat accepted this revision.Dec 31 2022, 4:53 AM

ashoat added inline comments.

native/account/siwe-panel.react.js
91 ↗	(On Diff #20462)	The string `'anonymous'` passed here doesn't actually get used for anything, does it?

This revision is now accepted and ready to land.Dec 31 2022, 4:53 AM

atul added inline comments.Dec 31 2022, 5:14 AM

native/account/siwe-panel.react.js
91 ↗	(On Diff #20462)	Nah, and based on discussion on Linear + further investigation it looks like the argument can be removed altogether. I'll replace "anonymous" with "PLACEHOLDER" until I get to https://linear.app/comm/issue/ENG-2601/remove-userid-arg-from-jsi-initializecryptoaccount to make it a bit more clear.

anonymous -> PLACEHOLDER

This revision was landed with ongoing or failed builds.Dec 31 2022, 5:20 AM

Closed by commit rCOMM1220a5821da4: [native] Generate `primaryIdentityPublicKey` and pass to `landing/SIWE`. · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM1220a5821da4: [native] Generate `primaryIdentityPublicKey` and pass to `landing/SIWE`.

Harbormaster completed remote builds in B14996: Diff 20469.Dec 31 2022, 5:30 AM

Revision Contents
Changeset List

Path

Size

native/

account/

siwe-panel.react.js

13 lines

Diff 20473

native/account/siwe-panel.react.js

Loading...