-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
feat(cast): add flag to disable HTTPS certificate validation for RPC #10869
Conversation
- 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.
79f8f2b
to
a501dcf
Compare
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.
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
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.
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
@grandizzy it's not a breaking change, sorry for the confusion. I'll fix the test. |
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
Resolve #10855
Solution
ProviderBuilder
andRuntimeTransport
to handle the acceptance of invalid certificates.Config
to includeeth_rpc_accept_invalid_certs
for managing this setting.PR Checklist