Skip to content

PYTHON-5677 Prevent ClientEncryption from loading crypt shared library #2659

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.

Sign up for GitHub

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

Jump to bottom
Open
aclark4life wants to merge 1 commit into
base: master
Choose a base branch
from
Open

PYTHON-5677 Prevent ClientEncryption from loading crypt shared library #2659

aclark4life wants to merge 1 commit into from
+11 −2

Conversation

@aclark4life
Copy link
Contributor

@aclark4life aclark4life commented Dec 19, 2025 •
edited
Loading

PYTHON-5677

Changes in this PR

  • Pass bypass_encryption=True to skip loading crypt_shared.

Test Plan

  • Rely on existing encryption tests to pass.

Checklist

Checklist for Author

  • Did you update the changelog (if necessary)?
  • Is there test coverage?
  • Is any followup work tracked in a JIRA ticket? If so, add link(s).

Checklist for Reviewer

  • Does the title of the PR reference a JIRA Ticket?
  • Do you fully understand the implementation? (Would you be comfortable explaining how this code works to someone else?)
  • Is all relevant documentation (README or docstring) updated?

@aclark4life aclark4life requested a review from a team as a code owner December 19, 2025 19:36
timgraham
timgraham reviewed Dec 19, 2025
View reviewed changes
@codeowners-service-app
Copy link

codeowners-service-app bot commented Dec 25, 2025

Assigned caseyclements for team dbx-python because NoahStapp is out of office.

caseyclements
caseyclements previously requested changes Dec 26, 2025
View reviewed changes
@timgraham timgraham changed the title PYTHON-5677: ClientEncryption shouldn't try to load mongo crypt shared library PYTHON-5677 Prevent ClientEncryption from loading crypt shared library Dec 26, 2025
@timgraham timgraham dismissed caseyclements’s stale review December 26, 2025 22:02

change addressed in another PR

@caseyclements caseyclements self-requested a review December 27, 2025 02:27
caseyclements
caseyclements approved these changes Dec 27, 2025
View reviewed changes
Copy link
Contributor

@caseyclements caseyclements left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@timgraham timgraham timgraham left review comments

@caseyclements caseyclements caseyclements approved these changes

@NoahStapp NoahStapp Awaiting requested review from NoahStapp NoahStapp is a code owner automatically assigned from mongodb/dbx-python

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@aclark4life @timgraham @caseyclements @kevinAlbs