Page MenuHomePhabricator

Ensure that the same ETH account is used for each signature
ClosedPublic

Authored by marcin on May 15 2024, 6:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 5:19 AM
Unknown Object (File)
Sat, Nov 2, 5:57 AM
Unknown Object (File)
Thu, Oct 31, 9:15 PM
Unknown Object (File)
Sat, Oct 26, 5:06 AM
Unknown Object (File)
Sat, Oct 26, 5:06 AM
Unknown Object (File)
Sat, Oct 26, 5:06 AM
Unknown Object (File)
Sat, Oct 26, 5:05 AM
Unknown Object (File)
Sat, Oct 26, 5:01 AM
Subscribers

Details

Summary

Ensure that signature takes no effect if user uses different account than the one they used for social proof generation.

Test Plan
  1. Log in
  2. Switch to different wallet in Rainbow app
  3. Generate backup message with diferent wallet.
  4. Ensure that alert is displayed and backup message generation screen doesn't go away.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

native/account/registration/missing-registration-data/missing-siwe-backup-message.react.js
32 ↗(On Diff #40218)

Please keep all lines to 80 chars. Also the period at the end here is unusual for invariants on this team

41–42 ↗(On Diff #40218)
native/account/registration/siwe-backup-message-creation.react.js
171–172 ↗(On Diff #40218)

Same here

Please address inline comments before landing

This revision is now accepted and ready to land.May 15 2024, 7:04 AM
  1. Address Ashoat's sugestions.
  2. Rebase before landing
This revision was landed with ongoing or failed builds.May 16 2024, 3:56 AM
This revision was automatically updated to reflect the committed changes.