]> granicus.if.org Git - python/commitdiff
Stripped whitespace from comma-separated values read from config file
authorVinay Sajip <vinay_sajip@yahoo.co.uk>
Thu, 14 Dec 2006 08:41:39 +0000 (08:41 +0000)
committerVinay Sajip <vinay_sajip@yahoo.co.uk>
Thu, 14 Dec 2006 08:41:39 +0000 (08:41 +0000)
Lib/logging/config.py

index f14eb122b7c7dae04158716d77ceca614ff0f242..11d2b7a79602f66638f0bf5a92b5ab82630cdbc7 100644 (file)
@@ -110,7 +110,7 @@ def _create_formatters(cp):
     flist = string.split(flist, ",")
     formatters = {}
     for form in flist:
-        sectname = "formatter_%s" % form
+        sectname = "formatter_%s" % string.strip(form)
         opts = cp.options(sectname)
         if "format" in opts:
             fs = cp.get(sectname, "format", 1)
@@ -139,7 +139,7 @@ def _install_handlers(cp, formatters):
     handlers = {}
     fixups = [] #for inter-handler references
     for hand in hlist:
-        sectname = "handler_%s" % hand
+        sectname = "handler_%s" % string.strip(hand)
         klass = cp.get(sectname, "class")
         opts = cp.options(sectname)
         if "formatter" in opts:
@@ -176,6 +176,7 @@ def _install_loggers(cp, handlers):
     # configure the root first
     llist = cp.get("loggers", "keys")
     llist = string.split(llist, ",")
+    llist = map(lambda x: string.strip(x), llist)
     llist.remove("root")
     sectname = "logger_root"
     root = logging.root
@@ -190,7 +191,7 @@ def _install_loggers(cp, handlers):
     if len(hlist):
         hlist = string.split(hlist, ",")
         for hand in hlist:
-            log.addHandler(handlers[hand])
+            log.addHandler(handlers[string.strip(hand)])
 
     #and now the others...
     #we don't want to lose the existing loggers,
@@ -225,7 +226,7 @@ def _install_loggers(cp, handlers):
         if len(hlist):
             hlist = string.split(hlist, ",")
             for hand in hlist:
-                logger.addHandler(handlers[hand])
+                logger.addHandler(handlers[string.strip(hand)])
 
     #Disable any old loggers. There's no point deleting
     #them as other threads may continue to hold references