-
Notifications
You must be signed in to change notification settings - Fork 612
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
Depend on aws-lc-rs
with default-features = false
#1768
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.
Nice!
Benchmark resultsInstruction countsSignificant differencesThere are no significant instruction count differences Other differencesClick to expand
Wall-timeSignificant differencesThere are no significant wall-time differences Other differencesClick to expand
Additional informationCheckout details:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1768 +/- ##
=======================================
Coverage 95.98% 95.98%
=======================================
Files 80 80
Lines 18209 18209
=======================================
Hits 17478 17478
Misses 731 731 ☔ View full report in Codecov by Sentry. |
That's curious. I would have thought we did, so I'm trying to convince myself why this isn't the case 🤔 It seems like we might have been using
Minor note: For folks trying to dig into the changeset, I think this latter feature is "ring-sig-verify" not "ring-cert-verify" |
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! I've convinced myself :-)
FWIW It looks like the same change is appropriate for webpki: rustls/webpki#224
rustls does not seem to need the ring-io or ring-sig-verify features, and omitting them avoids a dependency on an old version of the `untrusted` crate.
7495892
to
658391e
Compare
Arg, good catch. Fixed and force-pushed. |
rustls does not seem to need the ring-io or ring-sig-verify features,
and omitting them avoids a dependency on an old version of the
untrusted
crate.