[lib] Introduce `isValidPrimaryIdentityPublicKey`
ClosedPublic
Actions

Authored by atul on Dec 31 2022, 2:03 AM.

Details

Reviewers

ashoat
tomek

Commits

rCOMM423fcb8e9658: [lib] Introduce `isValidPrimaryIdentityPublicKey`

Summary

Going to use this in landing-handler to validate siwe-primary-identity-public-key header value before "injecting" into page.

Depends on D6124

Test Plan

unit tests

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Dec 31 2022, 2:03 AM

atul added inline comments.Dec 31 2022, 2:07 AM

lib/utils/siwe-utils.js
16 ↗	(On Diff #20467)	43 "base64 alphabet" characters, uses `+` and `/` instead of `-` and `_` based on the following key I generated: `98+/eB2MUVvYCpkESOS1zuWnWttsYWDKeDXl8T3o8LY`

atul published this revision for review.Dec 31 2022, 2:08 AM

Harbormaster completed remote builds in B14994: Diff 20467.Dec 31 2022, 2:17 AM

atul added a child revision: D6126: [keyserver] Include `siwePrimaryIdentityPublicKey` in `landingResponder` response.Dec 31 2022, 2:20 AM

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

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

rebase before landing

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

Closed by commit rCOMM423fcb8e9658: [lib] Introduce `isValidPrimaryIdentityPublicKey`. · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM423fcb8e9658: [lib] Introduce `isValidPrimaryIdentityPublicKey`.

Harbormaster completed remote builds in B14997: Diff 20470.Dec 31 2022, 5:30 AM

Revision Contents
Changeset List

Path

Size

lib/

utils/

siwe-utils.js

6 lines

siwe-utils.test.js

24 lines

Diff 20474

View Options

lib/utils/siwe-utils.js