Paths

Table of Contentst

[native] Modify `SIWEPanel` to hit `siwe_auth` endpoint once SIWE message is signed
ClosedPublic
Actions

Authored by atul on Dec 24 2022, 7:58 PM.

Details

Reviewers

ashoat
tomek

Commits

rCOMMc322654268c3: [native] Modify `SIWEPanel` to hit `siwe_auth` endpoint once SIWE message is…

Summary

Modify SIWEPanel to hit the siwe_auth endpoint instead of the create_account endpoint once SIWE message is signed.

We also modify the SIWE_AUTH_SUCCESS payload to a boolean indicating whether the signed message is valid and login/registration can continue.

NOTE: For now we will return from siweAuthResponder BEFORE login/registration... login/registration will come later at which point the payload for SIWE_AUTH_SUCCESS will look like RegisterResponse | LogInResponse or something.

Depends on D6026

Test Plan

Able to hit the endpoint and verify that the request payload is as expected:
https://blob.sh/58459b.png

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Dec 24 2022, 7:58 PM

atul edited the summary of this revision. (Show Details)Dec 24 2022, 8:00 PM

atul added a parent revision: D6027: [landing] Add `openConnectModal` truthiness check before calling function.

atul added a child revision: D6029: [keyserver] Introduce `isValidEthereumAddress`.

atul edited the summary of this revision. (Show Details)

Harbormaster completed remote builds in B14738: Diff 20102.Dec 24 2022, 8:12 PM

atul requested review of this revision.Dec 24 2022, 8:12 PM

atul edited the summary of this revision. (Show Details)Dec 25 2022, 3:45 PM

atul added a parent revision: D6026: [lib] Introduce `SIWEAuthServerCall` and use as argument for `siweAuth()`.

atul removed a parent revision: D6027: [landing] Add `openConnectModal` truthiness check before calling function.

Awesome!!

This revision is now accepted and ready to land.Dec 25 2022, 7:23 PM

rebase before addressing feedback

Harbormaster failed remote builds in B14759: Diff 20131!Dec 27 2022, 12:31 AM

address feedback

This revision was landed with ongoing or failed builds.Dec 27 2022, 12:40 AM

Closed by commit rCOMMc322654268c3: [native] Modify `SIWEPanel` to hit `siwe_auth` endpoint once SIWE message is… (authored by atul). · Explain Why

This revision was automatically updated to reflect the committed changes.

atul added a commit: rCOMMc322654268c3: [native] Modify `SIWEPanel` to hit `siwe_auth` endpoint once SIWE message is….

Harbormaster completed remote builds in B14760: Diff 20132.Dec 27 2022, 12:45 AM

Revision Contents
Changeset List

Path

Size

lib/

types/

redux-types.js

2 lines

native/

account/

siwe-panel.react.js

24 lines

Diff 20133

View Options

lib/types/redux-types.js