Paths

Table of Contentst

Differential D7048

[web] Use `signedIdentityKeysBlobSelector` in `SIWELoginForm`
ClosedPublic
Actions

Authored by atul on Mar 12 2023, 9:12 PM.

Tags

None

Referenced Files

	F424169: 1b82fd.png
	Mar 12 2023, 9:48 PM

	F424167: 9308e0.png
	Mar 12 2023, 9:48 PM

	F424168: Screenshot 2023-03-12 at 5.47.45 PM.png
	Mar 12 2023, 9:48 PM

Subscribers

None

Details

Reviewers

Commits

rCOMMe5072ffca7f7: [web] Use `signedIdentityKeysBlobSelector` in `SIWELoginForm`

Summary

Now that we have signedIdentityKeysBlobSelector, we can use it in TraditionalLoginForm (previous diff) and SIWELoginForm to simplify things.

Depends on D7047

Test Plan

Try to "SIWE" from web.

Screenshot 2023-03-12 at 5.47.45 PM.png (2×2 px, 731 KB)

Set breakpoint in siweAuthResponder and make sure signedIdentityKeysBlob comes through and is validated as expected.

9308e0.png (1×4 px, 1 MB)

Check the cookies table to see if signed_identity_keys column is populated correctly.

Diff Detail

Repository

Branch

arcpatch-D7048 (branched from master)

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Mar 12 2023, 9:12 PM

atul published this revision for review.Mar 12 2023, 9:14 PM

Harbormaster completed remote builds in B17262: Diff 23650.Mar 12 2023, 9:28 PM

update after fix + rebase

Harbormaster completed remote builds in B17264: Diff 23652.Mar 12 2023, 9:46 PM

atul edited the test plan for this revision. (Show Details)Mar 12 2023, 9:48 PM

atul added a child revision: D7049: [keyserver] Introduce `setCookieSignedIdentityKeysBlob`.Mar 12 2023, 10:36 PM

atul removed a reviewer: tomek.Mar 13 2023, 2:34 PM

ashoat accepted this revision.Mar 13 2023, 6:53 PM

ashoat added inline comments.

web/account/siwe-login-form.react.js
77–79 ↗	(On Diff #23652)	Should we get rid of this in favor of accessing the primary `ed25519` key via `signedIdentityKeysBlob`?

This revision is now accepted and ready to land.Mar 13 2023, 6:53 PM

rebase and land

Harbormaster completed remote builds in B17319: Diff 23720.Mar 14 2023, 4:45 PM

Closed by commit rCOMMe5072ffca7f7: [web] Use `signedIdentityKeysBlobSelector` in `SIWELoginForm`. · Explain WhyMar 14 2023, 4:45 PM

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMe5072ffca7f7: [web] Use `signedIdentityKeysBlobSelector` in `SIWELoginForm`.

atul added inline comments.Mar 14 2023, 4:47 PM

web/account/siwe-login-form.react.js
77–79 ↗	(On Diff #23652)	I think it makes sense to grab this from Redux directly. Instead of `stringify`ing in `signedIdentityKeysBlobSelector` and then `parse`ing the `primaryIdentityPublicKeys` out again.

Revision Contents
Changeset List

Path

Size

web/

account/

siwe-login-form.react.js

71 lines

Diff 23720

web/account/siwe-login-form.react.js

Loading...