Skip to content

kwargs are no longer passed to logging QueueHandler in 3.12 #118868

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

Closed
Kaundur opened this issue May 9, 2024 · 0 comments
Closed

kwargs are no longer passed to logging QueueHandler in 3.12 #118868

Kaundur opened this issue May 9, 2024 · 0 comments
Labels
type-bug An unexpected behavior, bug, or error

Comments

@Kaundur
Copy link
Contributor

Kaundur commented May 9, 2024

Bug report

Bug description:

kwargs are no longer passed to logging QueueHandler in 3.12

import logging
import logging.config
import logging.handlers
import queue


class TestQueueHandler(logging.handlers.QueueHandler):
    def __init__(self, *args, **kwargs):
        super().__init__(queue.Queue())

        print('test' in kwargs)
        # True in 3.11
        # False in 3.12


def main():
    config = {
        'version': 1,
        'handlers': {
            'default': {
                'class': TestQueueHandler,
                'test': '123'
            },
        },
        'loggers': {
            'a': {
                'handlers': ['default'],
            },
        }
    }

    logging.config.dictConfig(config)


if __name__ == '__main__':
    main()

CPython versions tested on:

3.11, 3.12

Operating systems tested on:

Linux

Linked PRs

@Kaundur Kaundur added the type-bug An unexpected behavior, bug, or error label May 9, 2024
Kaundur added a commit to Kaundur/cpython that referenced this issue May 9, 2024
Kaundur added a commit to Kaundur/cpython that referenced this issue May 13, 2024
vsajip added a commit that referenced this issue Jun 4, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jun 4, 2024
…-118869)

(cherry picked from commit dce14bb)

Co-authored-by: Kaundur <[email protected]>
Co-authored-by: Nice Zombies <[email protected]>
Co-authored-by: Vinay Sajip <[email protected]>
miss-islington pushed a commit to miss-islington/cpython that referenced this issue Jun 4, 2024
…-118869)

(cherry picked from commit dce14bb)

Co-authored-by: Kaundur <[email protected]>
Co-authored-by: Nice Zombies <[email protected]>
Co-authored-by: Vinay Sajip <[email protected]>
vsajip pushed a commit that referenced this issue Jun 4, 2024
vsajip pushed a commit that referenced this issue Jun 4, 2024
@vsajip vsajip closed this as completed Jun 4, 2024
barneygale pushed a commit to barneygale/cpython that referenced this issue Jun 5, 2024
noahbkim pushed a commit to hudson-trading/cpython that referenced this issue Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this issue Jul 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-bug An unexpected behavior, bug, or error
Projects
Development

No branches or pull requests

2 participants