Skip to content
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

chore!: Bump to rustls 0.22, tokio-rustls 0.25 #42

Merged
merged 4 commits into from
Mar 11, 2024
Merged

Conversation

fasterthanlime
Copy link
Collaborator

The API changed significanly, some knowledge about IV/salt/keys/etc. has disappeared from rustls proper so it has to be done here.

Also, we're affected by rustls/rustls#1833 for the time being, and tokio-rustls still isn't on rustls 0.23.x 🤷

@fasterthanlime fasterthanlime changed the title WIP: rustls 0.22 support chore!: Bump to rustls 0.22, tokio-rustls 0.25 Mar 11, 2024
Copy link

codecov bot commented Mar 11, 2024

Codecov Report

Attention: Patch coverage is 98.11321% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 88.05%. Comparing base (3283fbd) to head (746167e).

Files Patch % Lines
src/lib.rs 97.50% 2 Missing ⚠️
src/ffi.rs 98.73% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
+ Coverage   86.24%   88.05%   +1.80%     
==========================================
  Files           5        5              
  Lines         589      703     +114     
==========================================
+ Hits          508      619     +111     
- Misses         81       84       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fasterthanlime fasterthanlime added this pull request to the merge queue Mar 11, 2024
Merged via the queue into main with commit a3ec205 Mar 11, 2024
3 checks passed
@fasterthanlime fasterthanlime deleted the rustls-0.22 branch March 11, 2024 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant