Skip to content

fix(config): disable proxy detection in Etherscan client#14454

Open
AissataF wants to merge 1 commit intofoundry-rs:masterfrom
AissataF:proxy
Open

fix(config): disable proxy detection in Etherscan client#14454
AissataF wants to merge 1 commit intofoundry-rs:masterfrom
AissataF:proxy

Conversation

@AissataF
Copy link
Copy Markdown

@AissataF AissataF commented Apr 24, 2026

foundry-block-explorers defaults to reqwest::Client::default(), which triggers system proxy lookup via SCDynamicStore and crashes under macOS App Sandbox / Cursor IDE. Build a reqwest::Client with no_proxy() and pass it through with_client, Note: foundry-block-explorers pins reqwest = "0.12", so foundry-config picks up the same version to keep Client types compatible.

Copy link
Copy Markdown
Collaborator

@mablr mablr left a comment

Choose a reason for hiding this comment

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

Makes sense.

Maybe it would be cleaner to add no_proxy directly to foundry_block_explorers::ClientBuilder (in foundry-core repo) mirroring reqwest::ClientBuilder? (would remove reqwest dep here)

Comment thread crates/config/Cargo.toml
unit-prefix = "0.5.2"
rayon.workspace = true
regex.workspace = true
reqwest = { version = "0.12", default-features = false }
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

foundry_block_explorerswas bumped here 1ea7425 and now depends on 0.13

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

wait for foundry-rs/foundry-core#51 and fix them together.

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.

2 participants