Skip to content

feat(cast): add flag to disable HTTPS certificate validation for RPC #10869

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

Merged
merged 6 commits into from
Jun 28, 2025

Conversation

mablr
Copy link
Contributor

@mablr mablr commented Jun 28, 2025

Motivation

Resolve #10855

Solution

  • Updated ProviderBuilder and RuntimeTransport to handle the acceptance of invalid certificates.
  • Modified Config to include eth_rpc_accept_invalid_certs for managing this setting.
  • Adjusted related tests to ensure proper functionality of the new feature.

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes ( as default is false)

- Updated `ProviderBuilder` and `RuntimeTransport` to handle the acceptance of invalid certificates.
- Modified `Config` to include `eth_rpc_accept_invalid_certs` for managing this setting.
- Adjusted related tests to ensure proper functionality of the new feature.
@mablr mablr force-pushed the feature/toggle-rpc-cert-validation branch from 79f8f2b to a501dcf Compare June 28, 2025 09:17
Copy link
Member

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

hmm, I can see how this could be useful,

this feature isnt invasive because it's just another setting, so I dont have any strong opinions against this

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.

I too think this makes sense, needs fixing of default config test. Not sure why it is marked as breaking change as it should not change default behavior, could you please explain. Thank you

@mablr
Copy link
Contributor Author

mablr commented Jun 28, 2025

@grandizzy it's not a breaking change, sorry for the confusion.

I'll fix the test.

@grandizzy grandizzy self-requested a review June 28, 2025 16:43
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

@mattsse mattsse merged commit bc96621 into foundry-rs:master Jun 28, 2025
42 of 44 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Jun 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

feat(cast): add -k / --insecure flag to disable HTTPS certificate validation for cast calls
3 participants