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

Orbit verify #85

Merged
merged 11 commits into from
May 8, 2024
Merged

Orbit verify #85

merged 11 commits into from
May 8, 2024

Conversation

gvladika
Copy link
Contributor

@gvladika gvladika commented Apr 16, 2024

Adds script which does the source code verification of all the contracts deployed by the L1AtomicTokenBridgeCreator to the Orbit chain.

Script is applicable for the verifying source code on the Blockscout explorer. Steps are following:

  1. Update hardhat.config.ts. Find orbit field under networks and customChains and replace values with correct RPC and blockscout endpoints.
  2. yarn install && yarn build
  3. Set up .env - provide BASECHAIN_RPC , L1_TOKEN_BRIDGE_CREATOR (address of token bridge creator on parent chain) and INBOX_ADDRESS.
  4. Optionally provide the DEPLOYER_KEY. That's the private key of any funded address on the Orbit chain. It is required if you want to get UpgradeExecutor and aeWETH verified. Due to specifics of cross-chain deployment used by token bridge creator, the only way to get UpgradeExecutor and aeWETH verified is to deploy dummy instances on the Orbit chain and verify them. That way the original instances will get automatically verified because of the deployed bytecode match. If DEPLOYER_KEY is not provided, this step will be skipped.
  5. Run script as following: yarn run blockscout:verify --network orbit

@cla-bot cla-bot bot added the cla-signed label Apr 16, 2024
@gvladika gvladika requested review from gzeoneth and TucksonDev April 18, 2024 13:28
@gvladika gvladika marked this pull request as ready for review April 18, 2024 13:51
Copy link
Contributor

@TucksonDev TucksonDev left a comment

Choose a reason for hiding this comment

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

LGTM

@gvladika gvladika merged commit c9e1336 into main May 8, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants