Skip to content

Conversation

ctz
Copy link
Contributor

@ctz ctz commented Mar 6, 2025

rustls 0.21 is EOL for security support at the end of March 2025.

  • Drop rustls-pemfile, as it is now incorporated into rustls-pki-types; use the new iterator-based API.
  • Use the ring-based provider, to align with previous version's build-time requirements.

@isabelatkinson isabelatkinson changed the title Take rustls 0.23 RUST-2171 Update rustls to 0.23 Mar 6, 2025
Copy link
Contributor

@isabelatkinson isabelatkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ctz thank you for this PR! A few changes requested:

  • There is a failure in the CI lint task. Can you run cargo +1.84 clippy && cargo +1.84 clippy --all-features and fix? (see .evergreen/check-clippy.sh for the CI script)
  • The latest commit to main includes a fix to our MSRV task. Can you please merge/rebase so that we can run the MSRV task properly to check if this new version complies with our MSRV? If not, we may need to bump it.

I can also make and push these changes to your branch if you'd prefer.

rustls 0.21 is EOL for security support at the end of March 2025.

- Drop rustls-pemfile, as it is now incorporated into rustls-pki-types;
  use the new iterator-based API.
- Use the ring-based provider, to align with previous version's build-time
  requirements.
@ctz ctz force-pushed the jbp-rustls-0.23 branch from ac72082 to 02919e7 Compare March 10, 2025 13:43
@ctz
Copy link
Contributor Author

ctz commented Mar 10, 2025

Rebased and fixed the clippy lint.

Copy link
Contributor

@isabelatkinson isabelatkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small change requested, otherwise LGTM. Going to tag in the other member of the team for review as well.

@isabelatkinson isabelatkinson requested a review from abr-egn March 10, 2025 17:15
Copy link
Contributor

@abr-egn abr-egn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM modulo Isabel's proposed changes; thank you for contributing these updates!

ctz and others added 2 commits March 10, 2025 18:46
Copy link
Contributor

@isabelatkinson isabelatkinson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ctz thanks again for your contribution! I'll merge this once CI has finished running

@ctz
Copy link
Contributor Author

ctz commented Mar 10, 2025

Thanks for using rustls!

@isabelatkinson isabelatkinson merged commit 7b9280c into mongodb:main Mar 10, 2025
14 of 17 checks passed
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.

3 participants