-
Notifications
You must be signed in to change notification settings - Fork 534
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
feat: Support using rustls for TLS #491
Conversation
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.
Thanks for your contribution! This PR is excellent.
I left some comments on our public API, PTAL.
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.
Great!
@Xuanwo Thanks for merging! Could you please create a release for this? |
Let's go! |
Addressed in #497 |
That's much appreciated!
…On 02/08/2022 15:34, Xuanwo wrote:
Addressed in #497 <#497>
—
Reply to this email directly, view it on GitHub
<#491 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AABWF5UCVSH2NLV4IIVLAFDVXDFR5ANCNFSM55HORYWQ>.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I hereby agree to the terms of the CLA available at: https://databend.rs/dev/policies/cla/
Summary
This patch adds a feature flag
rustls
to userustls
instead ofopenssl
as TLS implementation.