Skip to content

gh-132106: Ensure that running logging.handlers.QueueListener cannot be started again #132444

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

Merged
merged 2 commits into from
Apr 13, 2025

Conversation

csm10495
Copy link
Contributor

@csm10495 csm10495 commented Apr 12, 2025

gh-132106 - Make it so start can't be called again while running
Helps prevents a thread leak


📚 Documentation preview 📚: https://cpython-previews--132444.org.readthedocs.build/

@csm10495 csm10495 requested a review from vsajip as a code owner April 12, 2025 16:31
@picnixz picnixz changed the title gh-132106 - Make it so start can't be called again while running gh-132106: Ensure that running logging.handlers.QueueListener cannot be started again Apr 12, 2025
@picnixz
Copy link
Member

picnixz commented Apr 12, 2025

This could be backported in some sense since it can be regarded as a bugfix, but I'll let @vsajip decide.

Copy link
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

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

Just some cosmetic improvements for rendering the method's name.

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@picnixz picnixz self-assigned this Apr 12, 2025
@vsajip vsajip merged commit 5863cd7 into python:main Apr 13, 2025
39 checks passed
@vsajip vsajip added the needs backport to 3.13 bugs and security fixes label Apr 13, 2025
@miss-islington-app
Copy link

Thanks @csm10495 for the PR, and @vsajip for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app
Copy link

Sorry, @csm10495 and @vsajip, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker 5863cd70b8782313b52bb8c71a4127d7ea4c50e9 3.13

@miss-islington-app miss-islington-app bot assigned vsajip and unassigned picnixz Apr 13, 2025
vsajip pushed a commit to vsajip/cpython that referenced this pull request Apr 13, 2025
…stener` cannot be started again (pythonGH-132444)

Prevents a thread leak
Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
(cherry picked from commit 5863cd7)

Co-authored-by: Charles Machalow <csm10495@gmail.com>
@bedevere-app
Copy link

bedevere-app bot commented Apr 13, 2025

GH-132471 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Apr 13, 2025
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