Paths

Table of Contentst

Differential D6129

[lib] Introduce `getSIWEStatementForPublicKey(...)`
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F3753347: D6129.diff
	Fri, Jan 10, 1:51 AM

	Unknown Object (File)
	Thu, Jan 2, 3:19 AM

	Unknown Object (File)
	Thu, Dec 26, 8:16 AM

	Unknown Object (File)
	Thu, Dec 26, 8:16 AM

	Unknown Object (File)
	Thu, Dec 26, 8:16 AM

	Unknown Object (File)
	Thu, Dec 26, 8:16 AM

	Unknown Object (File)
	Thu, Dec 26, 8:15 AM

	Unknown Object (File)
	Thu, Dec 26, 8:05 AM

Subscribers

None

Details

Reviewers

ashoat
tomek

Commits

rCOMM3142750b0b85: [lib] Introduce `getSIWEStatementForPublicKey(...)`

Summary

Given a public key, getSIWEStatementForPublicKey(...) generates a statement for SIWE message that conforms to what is described in the whitepaper:

https://blob.sh/a3d138.png

Depends on D6128

Test Plan

included a unit test

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

Harbormaster completed remote builds in B15001: Diff 20478.Dec 31 2022, 2:51 AM

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

atul added a parent revision: D6128: [landing] Add `siwePrimaryIdentityPublicKey` to `SIWEContext`.

atul added a child revision: D6130: [lib] Introduce `isValidSIWEStatementWithPublicKey(...)`.Dec 31 2022, 2:54 AM

ashoat accepted this revision.Jan 1 2023, 7:21 PM

ashoat added inline comments.

lib/utils/siwe-utils.js
48 ↗	(On Diff #20478)	Can we add a newline here?

This revision is now accepted and ready to land.Jan 1 2023, 7:21 PM

ashoat added inline comments.Jan 1 2023, 7:55 PM

lib/utils/siwe-utils.js
49 ↗	(On Diff #20478)	Should we just use `siweStatement` here to avoid copy-paste?

atul added inline comments.Jan 3 2023, 1:34 PM

lib/utils/siwe-utils.js
48 ↗	(On Diff #20478)	No, newline characters are prohibited in the `statement` field of the SIWE Message:
49 ↗	(On Diff #20478)	I'll make that change before landing. I initially decided against that to avoid too much redirection, but not a big deal.

rebase before addressing feedback

atul attached a referenced file: F315621: 0b38f4.png. (Show Details)Jan 3 2023, 1:35 PM

rebase after addressing feedback and before landing

This revision was landed with ongoing or failed builds.Jan 3 2023, 1:41 PM

Closed by commit rCOMM3142750b0b85: [lib] Introduce `getSIWEStatementForPublicKey(...)` (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMM3142750b0b85: [lib] Introduce `getSIWEStatementForPublicKey(...)`.

Harbormaster completed remote builds in B15053: Diff 20535.Jan 3 2023, 1:49 PM

Harbormaster completed remote builds in B15054: Diff 20536.Jan 3 2023, 1:53 PM

Revision Contents
Changeset List

Path

Size

lib/

utils/

11 lines

siwe-utils.test.js

11 lines

Diff 20537

lib/utils/siwe-utils.js

Loading...

lib/utils/siwe-utils.test.js

Loading...