validate metadata for communities with owner token #541
Conversation
🦋 Changeset detectedLatest commit: cbe8806 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
@felicio is attempting to deploy a commit to the Status Team on Vercel. A member of the Team first needs to authorize it. |
felicio
changed the title
| ['function getEntry(address _communityAddress) view returns (address)'], | ||
| this.#provider | ||
| ) | ||
| const ownerContractAddress = await registryContract.getEntry( |
There was a problem hiding this comment.
| ['function signerPublicKey() view returns (bytes)'], | ||
| this.#provider | ||
| ) | ||
| const owner = await ownerContract.signerPublicKey() |
There was a problem hiding this comment.
note: % cast call --rpc-url https://optimism-mainnet.infura.io/v3/<INFURA_API_KEY> 0x9eDc11E5932372387E76ff3dcF66DB5465893823 "signerPublicKey()" 0x0000000000000000000000000000000000000000000000000000000000000020000000000000000000000000000000000000000000000000000000000000004104953f5f0d355b37c39d1d6460a31ed1114455f8263b3fd1b84406c5f12c9eb7dfb76ba7513b92186010928254984fe98aee069b4c7e20f9ea3da497c3ae76947700000000000000000000000000000000000000000000000000000000000000
There was a problem hiding this comment.
The result should match message.signerPublicKey to consider the waku payload valid.
felicio
changed the title
|
@status-im/web ping |
| // return | ||
| // } | ||
| } else if (publicKey !== signerPublicKey) { | ||
| // todo?: zod |
There was a problem hiding this comment.
this is not coming from protobufs?
There was a problem hiding this comment.
What isn't?
The protobufs fields are optional by design.
|
@status-im/web ping |
| @@ -132,7 +132,7 @@ class Client { | |||
| } | |||
|
|
|||
| static async start(options: ClientOptions): Promise<Client> { | |||
| const { environment = 'test' } = options | |||
| // const { environment = 'development' } = options | |||
There was a problem hiding this comment.
ref #543 (comment)
No description provided.