Skip to content

Conversation

zerosnacks
Copy link
Member

@zerosnacks zerosnacks commented Aug 21, 2025

Motivation

Closes: #10913

Dependencies

Related

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 deprecated

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

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" }
Copy link
Member Author

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

@zerosnacks zerosnacks marked this pull request as ready for review August 28, 2025 14:36
onbjerg
onbjerg previously approved these changes Aug 28, 2025
Copy link
Collaborator

@onbjerg onbjerg left a 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

@zerosnacks zerosnacks marked this pull request as draft August 28, 2025 16:31
@zerosnacks
Copy link
Member Author

zerosnacks commented Aug 28, 2025

Temporarily moved back to draft, found something I want to address in block-explorers, specifically the removal of glueing the chain id given alloy-chains now correctly and always resolves to a chainid=<id> postfixed URL for Etherscan which requires it. Manually reviewed all URLs in alloy-rs/chains#196

@zerosnacks zerosnacks marked this pull request as ready for review August 29, 2025 13:15
@zerosnacks zerosnacks requested a review from onbjerg August 29, 2025 13:16
Copy link
Collaborator

@grandizzy grandizzy left a comment

Choose a reason for hiding this comment

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

lgtm!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

chore: fully deprecate Etherscan V1 for Etherscan
3 participants