Skip to content

switch sentry TLS to use rustls instead of native - #3627

Merged
GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:rustls
Oct 21, 2026
Merged

GuillaumeGomez merged 1 commit into
rust-lang:mainfrom
syphar:rustls

Conversation

@syphar

@syphar syphar commented Oct 21, 2026

Copy link
Copy Markdown
Member

will reduce dependencies a little when we dropped the last native user ( rustwide).

Also, is easier to manage security wise, when we only use one technique for root certs.

@syphar syphar self-assigned this Oct 21, 2026
@syphar
syphar requested a review from a team as a code owner October 21, 2026 07:29
@github-actions github-actions Bot added S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Oct 21, 2026
@syphar

syphar commented Oct 21, 2026

Copy link
Copy Markdown
Member Author

r? @GuillaumeGomez

@rustbot rustbot assigned GuillaumeGomez and unassigned syphar Oct 21, 2026
@GuillaumeGomez

Copy link
Copy Markdown
Member

Nice!

@GuillaumeGomez
GuillaumeGomez merged commit fecbc78 into rust-lang:main Oct 21, 2026
14 checks passed
@GuillaumeGomez
GuillaumeGomez deleted the rustls branch October 21, 2026 08:44
@github-actions github-actions Bot added S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it and removed S-waiting-on-review Status: This pull request has been implemented and needs to be reviewed labels Oct 21, 2026

This branch was successfully deployed

1 active deployment
github-pages — 11eddd5b Deployed Oct 21, 2026 by GuillaumeGomez via deploy #767
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-deploy This PR is ready to be merged, but is waiting for an admin to have time to deploy it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants