I made this suggestion sometime last year, but it never made it into the 2.68
Betas.
Allowing action_insert_header() [or action_add_header()] during filter_sender()
is technically necessary for the proper addition of SPF test results into a
message. The current stock source does not permit this operation. The patch
to allow this is trivial:
--- mimedefang.pl.in 2010-02-24 08:05:47.000000000 +0000
+++ mimedefang.pl.in 2010-02-24 08:06:50.000000000 +0000
@@ -1019,7 +1019,6 @@
sub action_insert_header ($$;$) {
my($header, $value, $pos) = @_;
$pos = 0 unless defined($pos);
- return if (!in_message_context('action_insert_header'));
write_result_line('N', $header, $pos, $value);
}
@@ -5736,8 +5735,10 @@
@ESMTPArgs = map { percent_decode($_) } split(' ', $_);
chdir($CWD);
+ open(RESULTS, ">>RESULTS");
sender_ok($sender, $ip, $name, $helo);
chdir($Features{'Path:SPOOLDIR'});
+ close(RESULTS);
next;
}
if ($_ =~ /^recipok
(\S*)\s+(\S*)\s+(\S*)\s+(\S*)\s+(\S*)\s+(\S*)\s+(\S*)\s+(\S*)\s+(\S+)\s+(\S+)\s+(\S+)/)
{
@@ -5939,7 +5940,7 @@
}
# Prepare the RESULTS file descriptor
- if (!open(RESULTS, ">RESULTS")) {
+ if (!open(RESULTS, ">>RESULTS")) {
fatal("$MsgID: Could not open RESULTS file: $!");
return -1;
}
Please make the patch into the stock MimeDefang. Thanks.
_______________________________________________
NOTE: If there is a disclaimer or other legal boilerplate in the above
message, it is NULL AND VOID. You may ignore it.
Visit http://www.mimedefang.org and http://www.roaringpenguin.com
MIMEDefang mailing list [email protected]
http://lists.roaringpenguin.com/mailman/listinfo/mimedefang
