related to #2586

Sending mails with the smtp_batch output might fail e.g. due to wrong
settings regarding the SMTP-Server. So far this is not reflected in the
exit code when running the bot via cli to trigger sending the mails.
Fixing this (at least for the non-interactive call) enables users to
integrate this into their monitoring properly.

Author: Lukas-Heindl

intelmq/bots/outputs/smtp_batch/output.py

@@ -167,8 +167,12 @@ def cli_run(self):
                 sys.exit(0)
             elif i == "all":
                 count = 0
+                exit_code = 0
                 for mail in mails:
-                    if self.build_mail(mail, send=True):
+                    succ = self.build_mail(mail, send=True)
+                    if not succ:
+                        exit_code = 1
+                    else:
                         count += 1
                         print(f"{mail.to} ", end="", flush=True)
                         try:
@@ -183,7 +187,7 @@ def cli_run(self):
                         if mail.path:
                             os.unlink(mail.path)
                 print(f"\n{count}× mail sent.\n")
-                sys.exit(0)
+                sys.exit(exit_code)
             elif i == "clear":
                 for mail in mails:
                     self.cache.redis.delete(mail.key)