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

Add the verifier contract address to docs for v0.10.0 #1727

Merged
merged 7 commits into from May 1, 2024

Conversation

nategraf
Copy link
Contributor

Add the verifier contract address to docs for v0.10.0

This comment was marked as outdated.

@pdg744
Copy link
Contributor

pdg744 commented Apr 29, 2024

Is v0.10.0 a typo in the PR name?

@pdg744
Copy link
Contributor

pdg744 commented Apr 29, 2024

Ohh, that's based on the numbering on risc0-ethereum.
Do we need/want to do anything to clarify that the versioning here is not related to the zkvm versioning?

| [RiscZeroGroth16Verifier.sol] | [Sepolia] | [0xe57255C10291771A2E14f7eb257555AE82D78302][sepolia-verifier] |
| Contract | Version | Network | Address |
| ----------------------------- | ---------------------------- | --------- | ---------------------------------------------------------------------- |
| [RiscZeroGroth16Verifier.sol] | [v0.9.0][contracts-v0.9.0] | [Sepolia] | [0xe57255C10291771A2E14f7eb257555AE82D78302][sepolia-verifier-v0.9.0] |
Copy link
Contributor

@pdg744 pdg744 Apr 29, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Currently clicking on either of the links in the contract column will land you on the v0.8 version of the contract.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As for the deployed contracts, I've realized that this is a "corporate needs you to find the difference between this contract and this contract" situation.... there were no changes to the verifier between 0.9 and 0.10. (Evidence git diff v0.9.0..v0.10.0 --stat -- contracts)

| Contract | Network | Address |
| ----------------------------- | --------- | -------------------------------------------------------------- |
| [RiscZeroGroth16Verifier.sol] | [Sepolia] | [0xe57255C10291771A2E14f7eb257555AE82D78302][sepolia-verifier] |
| Contract | Version | Network | Address |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we want a column here indicating the control root and/or list of allowed programs?
Or perhaps we want a separate table for tracking versioning for the control root?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I do like the simplicity of this table, so I'm hesitant to complicate it with the very technical implementation detail of the control root.

On the other hand, as a user, I'd like to have a way to confirm that the control root being used corresponds to the set of zkr programs I trust.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Best way for someone to check this is to look at the verified source code for the contract on Etherscan, pull the control root out of there, and verify it by recomputing it from the risc0 repo. We should indeed include instructions on how to do this, probably in a comment above the control root field in the contract source, and potentially elsewhere.

@nategraf nategraf enabled auto-merge (squash) May 1, 2024 21:03
@nategraf nategraf merged commit 88697a5 into main May 1, 2024
31 checks passed
@nategraf nategraf deleted the victor/update-verifier-contract-addr-v0.10.0 branch May 1, 2024 21:48
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.

None yet

3 participants