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

RUST-2144 Remove use of deprecated openssl_probe method #1294

Merged
merged 3 commits into from
Jan 30, 2025

Conversation

isabelatkinson
Copy link
Contributor

@isabelatkinson isabelatkinson changed the title RUST-2144 Remove use of deprecated openssl_probe method RUST-2144 Remove use of deprecated openssl_probe method Jan 30, 2025
@isabelatkinson isabelatkinson marked this pull request as ready for review January 30, 2025 16:46
@isabelatkinson isabelatkinson merged commit 2eb8d34 into mongodb:main Jan 30, 2025
8 of 11 checks passed
@isabelatkinson isabelatkinson deleted the fix-clippy branch January 30, 2025 17:29
@mmastrac
Copy link

@isabelatkinson @abr-egn As a heads up, this fix is not sufficient -- unfortunately this unsafe method call may still cause crashes if another thread is accessing the environment.

Please see the following examples for correct fixes which use an updated openssl API load_verify_locations:

@abr-egn
Copy link
Contributor

abr-egn commented Jan 31, 2025

@mmastrac Thanks for the heads up! I've opened #1297 to fix that.

isabelatkinson added a commit to isabelatkinson/mongo-rust-driver that referenced this pull request Feb 4, 2025
isabelatkinson added a commit that referenced this pull request Feb 4, 2025
* Fix compilation when `dns-resolver` is disabled (#1292)

* RUST-2144 Remove use of deprecated `openssl_probe` method (#1294)

* RUST-2144 Use recommended openssl probing API (#1297)

* RUST-2143 Test compiling with no default features enabled (#1296)

* release v3.2.1

---------

Co-authored-by: Abraham Egnor <[email protected]>
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