warn about incorrect notification backends, fix #95

SMTP is now also recognized as `smtp`backend. You may use multiple
notification backends (separated by comma). Defaults to `stdout`.

docs/docs/configuration/server.rst

@@ -43,7 +43,7 @@ session key and hostname. Here are the default values for this section:
     name =
     host =
     max-age = 15m
-    notify =
+    notify = stdout
 
 dbpath
     file location to the SQLite3 database, highly recommended to change this
@@ -76,8 +76,15 @@ max-age
     :ref:`Appendum: Timedelta <appendum-timedelta>` for valid values.
 
 notify
-    Select notification backend for new comments. Currently, only SMTP
+    Select notification backend(s) for new comments, separated by comma.
-    is available.
+    Available backends:
+
+    stdout :
+        Log to standard output. Default, if none selected.
+
+    smtp :
+        Send notifications via SMTP on new comments with activation (if
+        moderated) and deletion links.
 
 
 .. _CORS: https://developer.mozilla.org/en/docs/HTTP/Access_control_CORS

isso/__init__.py

@@ -92,10 +92,13 @@ class Isso(object):
         super(Isso, self).__init__(conf)
 
         subscribers = []
+        for backend in conf.getlist("general", "notify"):
+            if backend == "stdout":
                 subscribers.append(Stdout(None))
-
+            elif backend in ("smtp", "SMTP"):
-        if conf.get("general", "notify") == "smtp":
                 subscribers.append(SMTP(self))
+            else:
+                logger.warn("unknown notification backend '%s'", backend)
 
         self.signal = ext.Signal(*subscribers)
 

isso/core.py

@@ -95,7 +95,7 @@ class Config:
         "dbpath = /tmp/isso.db",
         "host = ",
         "max-age = 15m",
-        "notify = ",
+        "notify = stdout",
         "[moderation]",
         "enabled = false",
         "purge-after = 30d",