Page MenuHomePhorge

Fix signature of `Logger.debug()`
ClosedPublic

Authored by sicherha on Nov 6 2022, 6:21 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Mar 25, 2:42 PM
Unknown Object (File)
Mon, Mar 25, 1:39 AM
Unknown Object (File)
Sun, Mar 17, 9:47 PM
Unknown Object (File)
Sat, Mar 16, 11:01 PM
Unknown Object (File)
Fri, Mar 1, 9:52 AM
Unknown Object (File)
Feb 23 2024, 3:46 AM
Unknown Object (File)
Feb 20 2024, 2:29 PM
Unknown Object (File)
Jan 21 2024, 12:16 AM
Subscribers

Details

Summary

The default argument level=1 must not come before any positional
arguments; it must be accepted as a keyword argument.
As signified by the pylint comments, the signature previously did not
conform to the specification given in
https://docs.python.org/3/library/logging.html#logging.Logger.debug.

This commit fixes a longstanding problem with crashes occurring in
certain scenarios when the debug level is set to 9. One such crash could
be observed from within SQLAlchemy, which calls debug() in the
following way:
pool.logger.debug("Created new connection %r", connection)
That call caused pykolab's Logger.debug() method to interpret its
second argument as a numeric level, which resulted in a TypeError
because it was actually an sqlite3.Connection.

By accepting level as a keyword argument, all arguments are now
interpreted in the correct order.

Diff Detail

Repository
rP pykolab
Lint
Lint Not Applicable
Unit
Tests Not Applicable