Add config option to disable all log files

Author: linuxdaemon

cloudbot/__init__.py

@@ -23,6 +23,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")
 
@@ -49,25 +51,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"] = {
@@ -89,4 +95,5 @@ def _setup():
 
     logging.config.dictConfig(dict_config)
 
+
 _setup()

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

@@ -9,7 +9,6 @@
 # +---------+
 # | Formats |
 # +---------+
-from cloudbot.hook import Priority
 from cloudbot.util.formatting import strip_colors
 
 base_formats = {
@@ -127,6 +126,7 @@ def format_irc_event(event, args):
 
     return irc_default.format(server=event.conn.name, irc_raw=event.irc_raw)
 
+
 # +--------------+
 # | File logging |
 # +--------------+
@@ -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: