[Patches] [ python-Patches-1641790 ] logging leaks loggers

[SourceForge.net]

Patches item #1641790, was opened at 2007-01-22 17:00
Message generated for change (Comment added) made by therve
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1641790&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Library (Lib)
Group: Python 2.6
Status: Open
Resolution: None
Priority: 5
Private: No
Submitted By: TH (therve)
Assigned to: Nobody/Anonymous (nobody)
Summary: logging leaks loggers

Initial Comment:
In our application, we used to create a logger per client (to get IP/port automatically in the prefix). Unfortunately logging leaks loggers by keeping it into an internal dict (attribute loggerDict of Manager).

Attached a patch using a weakref object, with a test.

----------------------------------------------------------------------

>Comment By: TH (therve)
Date: 2007-01-22 17:09

Message:
Logged In: YES 
user_id=1038797
Originator: YES

Looking at the documentation, it seems keeping it is mandatory because you
must get the same instance with getLogger. Maybe it'd need a documented way
to remove from the dict, though.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1641790&group_id=5470