Repository navigation
fix(rust): use rustls for HTTPS - #712
Closed
Mandukhai-Alimaa wants to merge 4 commits into
Closed
Mandukhai-Alimaa wants to merge 4 commits into
Mandukhai-Alimaa wants to merge 4 commits into
Conversation
Mandukhai-Alimaa
added this pull request to stack #710
October 16, 2026 18:55
This was referenced Oct 16, 2026
Mandukhai-Alimaa
force-pushed
the
preserve-workflow-checks
branch
from
October 19, 2026 20:02
6e67c5f to
6c759e7
Compare
Mandukhai-Alimaa
force-pushed
the
rustls-migration
branch
from
October 19, 2026 20:02
478c757 to
e648ae9
Compare
Mandukhai-Alimaa
force-pushed
the
preserve-workflow-checks
branch
from
October 19, 2026 20:25
6c759e7 to
7df0298
Compare
Mandukhai-Alimaa
force-pushed
the
rustls-migration
branch
from
October 19, 2026 20:25
e648ae9 to
51108a9
Compare
Mandukhai-Alimaa
force-pushed
the
preserve-workflow-checks
branch
from
October 19, 2026 21:16
7df0298 to
e091c8c
Compare
Mandukhai-Alimaa
force-pushed
the
rustls-migration
branch
from
October 19, 2026 21:16
51108a9 to
fb19b35
Compare
Mandukhai-Alimaa
marked this pull request as ready for review
October 19, 2026 22:16
lidavidm
approved these changes
Oct 20, 2026
lidavidm
left a comment
Contributor
There was a problem hiding this comment.
Seems good, but do we need to update both the Go and Rust drivers? Maybe let's pick one to focus efforts on?
Mandukhai-Alimaa
force-pushed
the
preserve-workflow-checks
branch
from
October 20, 2026 16:54
e091c8c to
60d8514
Compare
Mandukhai-Alimaa
requested review from
eric-wang-1990,
gopalldb,
msrathore-db and
vikrantpuppala
as code owners
October 20, 2026 16:54
Mandukhai-Alimaa
removed this pull request from stack #710
October 20, 2026 16:58
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What's Changed
Use Rustls with native certificate roots to remove the Linux OpenSSL dependency. Preserve TLS options and declare the Windows system DLLs used by OAuth browser launch.
Validated: local build; CI tests, Clippy, builds, and package loading across Linux, macOS, and Windows; release dry run.
Stack 4/5: based on #711; followed by the validation update in #713.