Skip to content
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

Ship 3979 remove ws validation seth #1316

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

davidcauchi
Copy link
Contributor

@davidcauchi davidcauchi commented Nov 11, 2024

Add option to force SETH to initialize with HTTP instead of the default WS.

Intention is to avoid passing RPC urls with urls_secret and instead only set force_http=true


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes introduce configuration support for HTTP connections, improve readability, and ensure consistency in timeout settings across various network configurations. By allowing HTTP connections, users have more flexibility in how they interact with blockchain networks.

What

  • book/src/libs/seth.md
    • Added documentation on using HTTP connections by setting force_http to true, enhancing the guidance for users on connection options.
  • seth/config.go
    • Introduced a ForceHTTP boolean field in the Config struct to support HTTP connections, providing users with the ability to choose their preferred protocol.
  • seth/seth.toml
    • Added the force_http configuration option, enabling users to configure their connection protocol preference.
    • Standardized dial_timeout assignment across all network configurations to improve readability and maintain consistency.

@cl-sonarqube-production
Copy link

@davidcauchi davidcauchi merged commit 3b0f987 into main Nov 11, 2024
48 of 53 checks passed
@davidcauchi davidcauchi deleted the ship-3979-remove-ws-validation-seth branch November 11, 2024 10:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants