HomePhabricator
Diffusion Comm fe213e9f71eb

[lib] Require `primaryIdentityPublicKey` in `SIWEStatement`

Description

[lib] Require primaryIdentityPublicKey in SIWEStatement

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).

Reviewers: ashoat, tomek

Reviewed By: ashoat

Differential Revision: https://phab.comm.dev/D6826

Details

Provenance
atulAuthored on Feb 21 2023, 8:20 PM
Reviewer
ashoat
Differential Revision
D6826: [lib] Require `primaryIdentityPublicKey` in `SIWEStatement`
Parents
rCOMMd33e4c4fba27: [Keyserver/rust-addon] Call identity deleteUser on account deletion
Branches
Unknown
Tags
Unknown