Use STDERR as default log target

Author: amotl

CHANGES.rst

@@ -9,6 +9,7 @@ in progress
 - When title is not set in configuration settings, use ``mqttwarn: {topic}``
   instead of ``mqttwarn`` only. Thanks, Rob!
 - Add launch configuration for VSCode. Thanks, David!
+- Use STDERR as default log target
 
 
 2021-06-18 0.25.0

HANDBOOK.md

@@ -110,9 +110,19 @@ I've written an introductory post, explaining [what mqttwarn can be used for](ht
   cleansession = False
 
   ; logging
+
+  ; Log format
   logformat = '%(asctime)-15s %(levelname)-5s [%(module)s] %(message)s'
-  logfile   = 'mqttwarn.log'
+
+  ; Send log to STDERR (default)
+  logfile   = 'stream://sys.stderr'
 
+  ; Write log output to file
+  ; logfile   = 'mqttwarn.log'
+
+  ; Turn off logging completely
+  ; logfile   = false
+
   ; one of: CRITICAL, DEBUG, ERROR, INFO, WARN
   loglevel     = DEBUG
 

mqttwarn/commands.py

@@ -130,7 +130,10 @@ def setup_logging(config):
     LOGFORMAT = config.logformat
 
     # Send log messages to sys.stderr by configuring "logfile = stream://sys.stderr"
-    if LOGFILE.startswith('stream://'):
+    if not LOGFILE:
+        pass
+
+    elif LOGFILE.startswith('stream://'):
         LOGFILE = LOGFILE.replace('stream://', '')
         logging.basicConfig(stream=eval(LOGFILE), level=LOGLEVEL, format=LOGFORMAT)
 

mqttwarn/configuration.py

@@ -191,7 +191,6 @@ def load_configuration(configfile=None, name='mqttwarn'):
     defaults = {
         'clientid': name,
         'lwt': 'clients/{}'.format(name),
-        'logfile': os.getenv(name.upper() + 'LOG', name + '.log'),
     }
 
     return Config(configfile, defaults=defaults)