Skip to content

Conversation

@ljoss17
Copy link
Contributor

@ljoss17 ljoss17 commented Jan 20, 2025

Closes: #216

Following #203 which added a SignedStarknetHeader, this PR adds a public key field to the client state and uses the public key to verifies the header and signature in the SignedStarknetHeader

@ljoss17
Copy link
Contributor Author

ljoss17 commented Jan 23, 2025

For verify_membership_raw and verify_non_membership_raw,

fn verify_membership_raw(
&self,
prefix: &CommitmentPrefix,
proof: &CommitmentProofBytes,
root: &CommitmentRoot,
path: PathBytes,
value: Vec<u8>,
) -> Result<(), ClientError> {
Ok(())
}
fn verify_non_membership_raw(
&self,
prefix: &CommitmentPrefix,
proof: &CommitmentProofBytes,
root: &CommitmentRoot,
path: PathBytes,
) -> Result<(), ClientError> {
Ok(())
}
, we don't have access to the Context, thus to the secp256k1_verify method. One solution would be to use the secp256k1 library but this causes the Wasm contract to be too big to upload on Osmosis

@soareschen soareschen merged commit 6521d3b into main Jan 23, 2025
6 checks passed
@soareschen soareschen deleted the luca_joss/verify-relayer-cosmos-side branch January 23, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Verify Starknet header in Wasm light client

5 participants