Mercurial > p > roundup > code
comparison roundup/configuration.py @ 4337:9225a37fbeae
Fix file handle leak in some web interfaces with logging turned on
| author | Richard Jones <richard@users.sourceforge.net> |
|---|---|
| date | Mon, 01 Feb 2010 02:58:07 +0000 |
| parents | 88af08f8666f |
| children | 94c992852f12 |
comparison
equal
deleted
inserted
replaced
| 4336:aab5dba70839 | 4337:9225a37fbeae |
|---|---|
| 1286 hdlr = logging.StreamHandler(sys.stdout) | 1286 hdlr = logging.StreamHandler(sys.stdout) |
| 1287 formatter = logging.Formatter( | 1287 formatter = logging.Formatter( |
| 1288 '%(asctime)s %(levelname)s %(message)s') | 1288 '%(asctime)s %(levelname)s %(message)s') |
| 1289 hdlr.setFormatter(formatter) | 1289 hdlr.setFormatter(formatter) |
| 1290 # no logging API to remove all existing handlers!?! | 1290 # no logging API to remove all existing handlers!?! |
| 1291 for h in logger.handlers: | |
| 1292 h.close() | |
| 1293 logger.removeHandler(hdlr) | |
| 1291 logger.handlers = [hdlr] | 1294 logger.handlers = [hdlr] |
| 1292 logger.setLevel(logging._levelNames[self["LOGGING_LEVEL"] or "ERROR"]) | 1295 logger.setLevel(logging._levelNames[self["LOGGING_LEVEL"] or "ERROR"]) |
| 1293 | 1296 |
| 1294 def load(self, home_dir): | 1297 def load(self, home_dir): |
| 1295 """Load configuration from path designated by home_dir argument""" | 1298 """Load configuration from path designated by home_dir argument""" |