Skip to content

feat(client): support tls scheme#4

Open
IzioDev wants to merge 1 commit intokaspanet:mainfrom
IzioDev:feat/support-tls-scheme
Open

feat(client): support tls scheme#4
IzioDev wants to merge 1 commit intokaspanet:mainfrom
IzioDev:feat/support-tls-scheme

Conversation

@IzioDev
Copy link
Copy Markdown
Collaborator

@IzioDev IzioDev commented Feb 10, 2026

  • tighten tonic features to the required minimum
  • validate provided URI before initializing the tonic client. additionally: add opinion on how URI should be constructed regarding authority - it must be present, URI like /my-path aren't supported (not a regression).
  • allow more valid URI forms: 127.0.0.1, http://127.0.0.1, https://example.com, http://user:password:port/my/path?arg=1
  • add TLS support, using the native certificate store, on this tonic version (0.12.3) the provider is ring (crate) and it seems there is no way to change it. on newer versions, one can chose among the provided ones or provide a custom.
  • grpc:// usage has been removed as it's non std

@IzioDev
Copy link
Copy Markdown
Collaborator Author

IzioDev commented Feb 12, 2026

For some reasons, I can't tag you @supertypo to request a review from you.
No urge at all, but this message act as a review request :)

@supertypo
Copy link
Copy Markdown
Collaborator

Sorry for taking my time with this. I'll QA shortly

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