PYTHON-5867 - Close sockets on interruption or cancellation during async connection creation

[NoahStapp]

PYTHON-5867

Changes in this PR

Close sockets opened as part of async connection creation when an interruption or cancellation occurs.

Test Plan

Fix existing PyPy test failure.

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?

[Copilot]

Pull request overview

This PR addresses a resource-leak scenario in PyMongo’s async connection setup by ensuring sockets/transports are closed when connection creation is interrupted (e.g., task cancellation), which is especially important for event-loop reliability and preventing leaked file descriptors.

Changes:

• Add cancellation/interruption-safe cleanup (except BaseException: close(); raise) in async socket creation and TLS configuration paths.
• Ensure transports are aborted / sockets are closed when failures occur during async protocol interface setup.
• Add a destructor (__del__) to AsyncNetworkingInterface to synchronously close the underlying raw socket if the connection is orphaned (e.g., loop already closed).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
pymongo/pool_shared.py	Adds broad-exception cleanup to prevent socket leaks during async connection creation/configuration and protocol setup.
pymongo/network_layer.py	Adds AsyncNetworkingInterface.__del__ to defensively close raw sockets when async connections are orphaned.

[codecov-commenter]

Codecov Report

❌ Patch coverage is 65.00000% with 7 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
pymongo/pool_shared.py	65.00%	5 Missing and 2 partials ⚠️

📢 Thoughts on this report? Let us know!

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

[sophiayangDB]

LGTM!

[aclark4life]

Do _socket and _ssl need to be private here ?

[NoahStapp]

Good catch, vestigial import naming from an earlier version that overrode them.