aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--driver.c19
1 files changed, 18 insertions, 1 deletions
diff --git a/driver.c b/driver.c
index dea10ba5..2a45bfcb 100644
--- a/driver.c
+++ b/driver.c
@@ -1254,6 +1254,9 @@ int num; /* index of message */
* This is a potential problem if the MTAs further upstream
* didn't pass canonicalized From/Return-Path lines, *and* the
* local SMTP listener insists on them.
+ *
+ * None of these error conditions generates bouncemail. Comments
+ * below explain for each case why this is so.
*/
ap = (return_path[0]) ? return_path : user;
if (SMTP_from(ctl->smtp_socket, ap, options) != SM_OK)
@@ -1272,6 +1275,10 @@ int num; /* index of message */
* 571 = sendmail's "unsolicited email refused"
* 501 = exim's old antispam response
* 550 = exim's new antispam response (temporary)
+ *
+ * We don't send bouncemail on antispam failures because
+ * we don't want the scumbags to know the address is even
+ * valid.
*/
SMTP_rset(ctl->smtp_socket); /* required by RFC1870 */
free(headers);
@@ -1294,7 +1301,12 @@ int num; /* index of message */
* Temporary out-of-queue-space condition on the
* ESMTP server. Don't try to ship the message,
* and suppress deletion so it can be retried on
- * a future retrieval cycle.
+ * a future retrieval cycle.
+ *
+ * Bouncemail *might* be appropriate here as a delay
+ * notification. But it's not really necessary because
+ * this is not an actual failure, we're very likely to be
+ * able to recover on the next cycle.
*/
SMTP_rset(ctl->smtp_socket); /* required by RFC1870 */
free(headers);
@@ -1307,6 +1319,11 @@ int num; /* index of message */
* Permanent no-go condition on the
* ESMTP server. Don't try to ship the message,
* and allow it to be deleted.
+ *
+ * Bouncemail would be appropriate for 552, but in these
+ * latter days 553 usually means a spammer is trying to
+ * cover his tracks. We'd rather deny the scumbags any
+ * feedback that the address is valid.
*/
SMTP_rset(ctl->smtp_socket); /* required by RFC1870 */
free(headers);