Removing logging in favor of just throwing. Removing user from message as it doesn't add any value.

config/action.d/smtp.py

@@ -165,10 +165,7 @@ class SMTPAction(ActionBase):
 			if self.ssl: # pragma: no cover
 				tls_result = smtp.starttls()[0];
 				if tls_result != 220: # pragma: no cover
-					self._logSys.error(
+					raise Exception("Failed to starttls() on '%s': %s" % (self.host, tls_result))
-						"Failed to starttls() on '%s' for user '%s': %s",
-						self.host, self.user, tls_result)
-					raise Exception("Failed to starttls()")
 
 			if self.user and self.password: # pragma: no cover (ATM no tests covering that)
 				smtp.login(self.user, self.password)