-
Notifications
You must be signed in to change notification settings - Fork 2.1k
chore: deprecate Etherscan V1 #11387
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
base: master
Are you sure you want to change the base?
Conversation
Cargo.toml
Outdated
@@ -407,7 +407,7 @@ idna_adapter = "=1.1.0" | |||
# revm-inspectors = { git = "https://github.com/zerosnacks/revm-inspectors.git", rev = "74e215d" } | |||
|
|||
## foundry | |||
# foundry-block-explorers = { git = "https://github.com/foundry-rs/block-explorers.git", rev = "e09cb89" } | |||
foundry-block-explorers = { git = "https://github.com/foundry-rs/block-explorers.git", rev = "80ff600" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Patch to be removed after new release of block-explorers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cant argue with a -200 delta
Temporarily moved back to draft, found something I want to address in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
Motivation
Closes: #10913
Dependencies
chain
, the URL is available directly and correctly inalloy-chains
. block-explorers#109Related
Solution
Fully deprecates Etherscan version abstraction in favor of simpler V2 implementation as Etherscan has officially fully deprecated support for V1.
Manually tested against Etherscan, Blockscout and Sourcify without issue. No changes are actually expected considering the Etherscan V2 API is backwards compatible with Etherscan V1.
Breaking changes
The
etherscan_api_version
field and setting are deprecatedPR Checklist