From dd16e114db70592df34e8b9ee4a05dae69aaba43 Mon Sep 17 00:00:00 2001 From: Philip Hazel Date: Thu, 20 Apr 2006 10:57:46 +0000 Subject: [PATCH] Modify (i.e. fix :-) change 4.61/PH/19: Using message arrival time for "first failed" should *not* apply to host errors. --- doc/doc-txt/ChangeLog | 7 ++++++- src/src/retry.c | 15 ++++++++------- test/log/0532 | 2 +- test/rejectlog/0532 | 2 +- test/stderr/0388 | 3 ++- 5 files changed, 18 insertions(+), 11 deletions(-) diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog index 9f8d80b14..f51f97071 100644 --- a/doc/doc-txt/ChangeLog +++ b/doc/doc-txt/ChangeLog @@ -1,4 +1,4 @@ -$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.342 2006/04/19 11:15:43 ph10 Exp $ +$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.343 2006/04/20 10:57:46 ph10 Exp $ Change log file for Exim from version 4.21 ------------------------------------------- @@ -24,6 +24,10 @@ PH/02 When tcp-wrappers is called from Exim, it returns only "deny" or "allow". PH/03 Add -lutil to the default FreeBSD LIBS setting. +PH/04 Change PH/19 for 4.61 was too wide. It should not be applied to host + errors. Otherwise a message that provokes a temporary error (when other + messages do not) can cause a whole host to time out. + Exim version 4.61 ----------------- @@ -137,6 +141,7 @@ PH/19 When calculating a retry time, Exim used to measure the "time since when small messages continue to be delivered. Without this change, the "time since failure" will always be short, possible causing more frequent delivery attempts for the huge message than are intended. + [Note: This change was subsequently modified - see PH/04 for 4.62.] PH/20 Added $auth1, $auth2, $auth3 to contain authentication data (as well as $1, $2, $3) because the numerical variables can be reset during some diff --git a/src/src/retry.c b/src/src/retry.c index ca61e5c0c..afb40ef90 100644 --- a/src/src/retry.c +++ b/src/src/retry.c @@ -1,4 +1,4 @@ -/* $Cambridge: exim/src/src/retry.c,v 1.9 2006/03/09 15:10:16 ph10 Exp $ */ +/* $Cambridge: exim/src/src/retry.c,v 1.10 2006/04/20 10:57:46 ph10 Exp $ */ /************************************************* * Exim - an Internet mail transport agent * @@ -698,13 +698,14 @@ for (i = 0; i < 3; i++) DEBUG(D_retry) debug_printf("failing_interval=%d message_age=%d\n", failing_interval, message_age); - /* If the message has been on the queue longer than the recorded time - of failure, use the message's age instead. This can happen when some - messages can be delivered and others cannot; a successful delivery will - reset the first_failed time, and this can lead to a failing message - being retried too often. */ + /* For a non-host error, if the message has been on the queue longer + than the recorded time of failure, use the message's age instead. This + can happen when some messages can be delivered and others cannot; a + successful delivery will reset the first_failed time, and this can lead + to a failing message being retried too often. */ - if (message_age > failing_interval) failing_interval = message_age; + if ((rti->flags & rf_host) == 0 && message_age > failing_interval) + failing_interval = message_age; /* Search for the current retry rule. The cutoff time of the last rule is handled differently to the others. The rule continues diff --git a/test/log/0532 b/test/log/0532 index 57a152060..0a41abcdb 100644 --- a/test/log/0532 +++ b/test/log/0532 @@ -5,4 +5,4 @@ 1999-03-02 09:44:33 10HmaY-0005vi-00 <= CALLER@test.ex U=CALLER P=local S=sss 1999-03-02 09:44:33 10HmaY-0005vi-00 => rcptok R=r1 T=t1 1999-03-02 09:44:33 10HmaY-0005vi-00 Completed -1999-03-02 09:44:33 U=CALLER temporarily rejected connection in "connect" ACL: cannot test add_header condition in connection ACL +1999-03-02 09:44:33 U=CALLER temporarily rejected connection in "connect" ACL: cannot use add_header condition in connection ACL diff --git a/test/rejectlog/0532 b/test/rejectlog/0532 index 7a20d074e..5fe9ed425 100644 --- a/test/rejectlog/0532 +++ b/test/rejectlog/0532 @@ -1,2 +1,2 @@ 1999-03-02 09:44:33 U=CALLER F= rejected RCPT -1999-03-02 09:44:33 U=CALLER temporarily rejected connection in "connect" ACL: cannot test add_header condition in connection ACL +1999-03-02 09:44:33 U=CALLER temporarily rejected connection in "connect" ACL: cannot use add_header condition in connection ACL diff --git a/test/stderr/0388 b/test/stderr/0388 index 0af0261ba..ab7d8ae32 100644 --- a/test/stderr/0388 +++ b/test/stderr/0388 @@ -156,8 +156,9 @@ V4NET.0.0.0 in "*"? yes (matched "*") retry for T:V4NET.0.0.0:V4NET.0.0.0:1224 (y) = * 0 0 dbfn_read: key=T:V4NET.0.0.0:V4NET.0.0.0:1224 failing_interval=ttt message_age=ttt +on queue longer than maximum retry Writing retry data for T:V4NET.0.0.0:V4NET.0.0.0:1224 - first failed=dddd last try=dddd next try=+1 expired=1 + first failed=dddd last try=dddd next try=+0 expired=0 errno=dd more_errno=dd,A Network Error dbfn_write: key=T:V4NET.0.0.0:V4NET.0.0.0:1224 timed out: all retries expired -- 2.30.2