multiprocessing-logging icon indicating copy to clipboard operation
multiprocessing-logging copied to clipboard

uninstall_mp_handler() doesn't handle >1 handlers

Open knightofni-i opened this issue 5 years ago • 2 comments
trafficstars

Need to use for handler in list(logger.handlers): instead of current for handler in logger.handlers: When I get some time, I will fork project and make a pull request.

knightofni-i avatar Sep 01 '20 16:09 knightofni-i

Additionally, we should call self.manager.shutdown() in MultiProcessingHandler.close()

knightofni-i avatar Sep 01 '20 16:09 knightofni-i

This was contributed and has no tests. I'll work on it eventually. Alternatively, if you want, you can submit a PR with tests and the fix.

jruere avatar Sep 09 '20 06:09 jruere

This was fixed without tests. Will be released on the next version.

jruere avatar Dec 21 '22 18:12 jruere