Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(tee-key-preexec): support for onchain-compatible pubkey in report_data #251

Merged
merged 2 commits into from
Jan 17, 2025

Conversation

pbeza
Copy link
Contributor

@pbeza pbeza commented Jan 15, 2025

This PR is part of the effort to implement on-chain TEE proof verification. This PR goes hand in hand with matter-labs/zksync-era#3414.

…report_data

This PR is part of the effort to implement on-chain TEE proof
verification. This PR goes hand in hand with matter-labs/zksync-era#3414.
@pbeza pbeza force-pushed the pab/onchain-verification branch from e6fc60d to 2d04ba0 Compare January 16, 2025 19:46
crates/teepot/src/prover/reportdata.rs Show resolved Hide resolved
bin/tee-key-preexec/src/main.rs Outdated Show resolved Hide resolved
bin/verify-attestation/src/main.rs Outdated Show resolved Hide resolved
@pbeza
Copy link
Contributor Author

pbeza commented Jan 17, 2025

I also removed this test as it is a duplicate of:

// Decode the sample secret key, generate the public key, and derive the Ethereum address
// from the public key
let secp = Secp256k1::new();
let secret_key_bytes =
hex::decode("c87509a1c067bbde78beb793e6fa76530b6382a4c0241e5e4a9ec0a0f44dc0d3")
.unwrap();
let secret_key = SecretKey::from_slice(&secret_key_bytes).unwrap();
let public_key = PublicKey::from_secret_key(&secp, &secret_key);
let expected_address = hex::decode("627306090abaB3A6e1400e9345bC60c78a8BEf57").unwrap();
let address = public_key_to_ethereum_address(&public_key);
assert_eq!(address, expected_address.as_slice());

Ready to merge. 🫡

@haraldh haraldh merged commit e2c3191 into main Jan 17, 2025
14 checks passed
@haraldh haraldh deleted the pab/onchain-verification branch January 17, 2025 12:17
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.

2 participants