Merge pull request CloudBotIRC#131 from linuxdaemon/gonzobot+file-log-config

Add config option to disable all log files

Author: edwardslabs

cloudbot/__init__.py

@@ -24,6 +24,8 @@ def _setup():
     else:
         logging_config = {}
 
+    file_log = logging_config.get("file_log", False)
+
     global logging_dir
     logging_dir = os.path.join(os.path.abspath(os.path.curdir), "logs")
 
@@ -50,25 +52,29 @@ def _setup():
                 "formatter": "brief",
                 "level": "INFO",
                 "stream": "ext://sys.stdout"
-            },
-            "file": {
-                "class": "logging.handlers.RotatingFileHandler",
-                "maxBytes": 1000000,
-                "backupCount": 5,
-                "formatter": "full",
-                "level": "INFO",
-                "encoding": "utf-8",
-                "filename": os.path.join(logging_dir, "bot.log")
             }
         },
         "loggers": {
             "cloudbot": {
                 "level": "DEBUG",
-                "handlers": ["console", "file"]
+                "handlers": ["console"]
             }
         }
     }
 
+    if file_log:
+        dict_config["handlers"]["file"] = {
+            "class": "logging.handlers.RotatingFileHandler",
+            "maxBytes": 1000000,
+            "backupCount": 5,
+            "formatter": "full",
+            "level": "INFO",
+            "encoding": "utf-8",
+            "filename": os.path.join(logging_dir, "bot.log")
+        }
+
+        dict_config["loggers"]["cloudbot"]["handlers"].append("file")
+
     if logging_config.get("console_debug", False):
         dict_config["handlers"]["console"]["level"] = "DEBUG"
         dict_config["loggers"]["asyncio"] = {

config.default.json

@@ -120,6 +120,7 @@
         "show_plugin_loading": true,
         "show_motd": true,
         "show_server_info": true,
-        "raw_file_log": false
+        "raw_file_log": false,
+        "file_log": true
     }
 }

plugins/log.py

@@ -219,6 +219,10 @@ def log(event):
     """
     :type event: cloudbot.event.Event
     """
+    logging_config = event.bot.config.get("logging", {})
+    if not logging_config.get("file_log", False):
+        return
+
     text = format_event(event)
 
     if text is not None: