Skip to content

[pull] master from MagicStack:master#23

Merged
pull[bot] merged 3 commits intoddelange:masterfrom
MagicStack:master
Oct 11, 2025
Merged

[pull] master from MagicStack:master#23
pull[bot] merged 3 commits intoddelange:masterfrom
MagicStack:master

Conversation

@pull
Copy link

@pull pull bot commented Oct 11, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

AndrewJackson2020 and others added 3 commits October 10, 2025 22:14
If _can_use_connection fails (say, because of an asyncio timeout), then we may have a full connection that has been created but would be leaked by this function. (note: `_connect_addr` takes care of cleaning up after itself if it fails partway)

This is particularly possible in the case of pgbouncer, where we may succeed at establishing a connection much quicker than even a trivial call to the backing database would take.

I believe this failure mode was introduced in #987

I'm not certain if we should `await` here or just punt the closing of the other connections to a background task (I don't know how risky `close` is on an established connection).
GitHub Windows runners somehow got slower over time.
@pull pull bot locked and limited conversation to collaborators Oct 11, 2025
@pull pull bot added the ⤵️ pull label Oct 11, 2025
@pull pull bot merged commit b4d2892 into ddelange:master Oct 11, 2025
40 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants