[back on list]
Brian C. Huffman wrote:
On Tue, 2007-01-23 at 16:55 -0500, Daryl C. W. O'Shea wrote:
Brian C. Huffman wrote:
Looking through /usr/sbin/amavisd, it appears that it *does not* add the
Received header if it is called via a milter (which it is). So, this wouldn't
seem to be the issue.
New versions do. The old versions that don't are broken and have
greater issues (most DNSBL tests and other stuff is broken) than
INVALID_TZ_EST firing.
Daryl
Why wouldn't sendmail itself insert the proper Received line and then
pass to amavisd (via the milter)?
The milter interface passes the headers and body as received. It does
not first prepend the received header for the local hop.
Here are a couple of lines
from /usr/sbin/amavisd that suggested to me that it wasn't inserting a
header:
Line 573:
$insert_received_line = 1; # insert Received: header field? (not with
milter)
Now that could mean that they think you should set it to "0" if you're
doing a milter although it sounds like you're saying that's not the
correct way.
I have no idea what the "not with milter" comment means, or the context
that code is in. I've never looked at, or downloaded, the amavisd
source... I'm having a good day if I spell amavisd correctly.
And then again this group of lines looks to me like it won't add the
received line if delivery_method = '' (which it would for milter,
wouldn't it)?
# misnomer, this _is_ the Received line
$hdr_edits->append_header_above_received('Received',
received_line($conn,$msginfo,$msginfo->mail_id,1), 1)
if c('insert_received_line') && $msginfo->delivery_method ne '' &&
$allowed_hdrs && $allowed_hdrs->{lc('Received')};
$hdr_edits;
}
and again here which does say that if it's implicit delivery,
deliver_method='':
sub delivery_method # delivery method, or empty for implicit delivery
(milter)
{ my($self)=shift; [EMAIL PROTECTED] ?
$self->{deliv_method}:($self->{deliv_method}=shift)}
I'm not sure exactly what Received header this is referring to. Does
amavisd add some sort of "Received from localhost/wherever by amavisd"?
If it's not some "Received ... by amavisd" header, I can't think of a
reason why it would be adding Received headers EXCEPT for when it's
acting as a milter.
I do appreciate your help, I'm just trying to understand how this all
works.
You are running the latest version of amavisd, correct? I know for
*sure* that a milter must fake a received header before passing the
message to SA and I'm 99.9% sure that it was amavisd that wasn't doing
this only a couple of months ago.
Of course, there's a good possibility that it's not related to amavisd
(since nobody else seems to be seeing the INVALID_TZ_EST hits), but
given your simple test message passed through, the only other thing I
can think of is some screwed up timezone setting in your MTA config.
Daryl
