Page MenuHomePhabricator

[lib] Require `primaryIdentityPublicKey` in `SIWEStatement`
ClosedPublic

Authored by atul on Feb 21 2023, 8:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 3, 1:47 PM
Unknown Object (File)
Wed, Feb 26, 11:19 AM
Unknown Object (File)
Fri, Feb 14, 6:35 PM
Unknown Object (File)
Tue, Feb 11, 7:30 AM
Unknown Object (File)
Feb 6 2025, 5:56 AM
Unknown Object (File)
Feb 3 2025, 5:55 AM
Unknown Object (File)
Feb 2 2025, 3:06 PM
Unknown Object (File)
Jan 10 2025, 2:46 PM
Subscribers
None

Details

Summary

We were previously accepting SIWEMessages without primaryIdentityPublicKey before we were able to integrate olm into web. Now that we've been able to integrate olm into web and are always including the primaryIdentityPublicKey in the SIWEMessage statement, we can validate that it's being included.

We don't have to worry about native clients that aren't including primaryIdentityPublicKey because none were publicly released.

Test Plan

Able to SIWE on both web and native without issue (against local keyserver).

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable