-
Notifications
You must be signed in to change notification settings - Fork 13
Replace rustls-native-certs by rustls-platform-verifier #1035
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.
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
base: main
Are you sure you want to change the base?
Conversation
BenchmarksComparisonBenchmark execution time: 2025-04-22 17:24:24 Comparing candidate commit 2926133 in PR branch Found 8 performance improvements and 8 performance regressions! Performance is the same for 36 metrics, 2 unstable metrics. scenario:credit_card/is_card_number/ 3782-8224-6310-005
scenario:credit_card/is_card_number/ 378282246310005
scenario:credit_card/is_card_number/378282246310005
scenario:normalization/normalize_name/normalize_name/bad-name
scenario:normalization/normalize_name/normalize_name/good
scenario:normalization/normalize_service/normalize_service/A0000000000000000000000000000000000000000000000000...
scenario:normalization/normalize_service/normalize_service/Test Conversion 0f Weird !@#$%^&**() Characters
scenario:normalization/normalize_service/normalize_service/test_ASCII
CandidateCandidate benchmark detailsGroup 1
Group 2
Group 3
Group 4
Group 5
Group 6
Group 7
Group 8
Group 9
Group 10
Group 11
Group 12
Group 13
BaselineOmitted due to size. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1035 +/- ##
==========================================
+ Coverage 71.55% 71.57% +0.01%
==========================================
Files 339 339
Lines 50668 50668
==========================================
+ Hits 36258 36266 +8
+ Misses 14410 14402 -8
🚀 New features to boost your workflow:
|
203a58e
to
978894a
Compare
Artifact Size Benchmark Reportaarch64-alpine-linux-musl
aarch64-unknown-linux-gnu
libdatadog-x64-windows
libdatadog-x86-windows
x86_64-alpine-linux-musl
x86_64-unknown-linux-gnu
|
f6fc94f
to
2926133
Compare
What does this PR do?
Replace
rustls-native-certs
byrustls-platform-verifier
as recommended in native-certs docMotivation
rustls-native-certs
is causing some issues when dropping the tokio runtime to prepare for forking on macos.Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.