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.

Sign up for GitHub

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

Jump to bottom

Add tls config to client #1158

Merged
merged 1 commit into from
May 27, 2022
Merged

Add tls config to client #1158

merged 1 commit into from
May 27, 2022

Conversation

mfridman
Copy link
Member

This re-adds NewConfig and passes the config.TLS to the http client.

97f8a3e#diff-73cbaebe6766a371dfca1849d8942ec87a6e1e7b808634e5d97a3d146599770aL821-L824

@mfridman mfridman merged commit 61010f9 into main May 27, 2022
@mfridman mfridman deleted the config-tls branch May 27, 2022 18:46
Monirul1 pushed a commit to Monirul1/buf that referenced this pull request Apr 30, 2023
@mfridman
Add optional tls config to client (bufbuild#1158)
8966a95
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smaye81 smaye81 smaye81 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mfridman @smaye81