Sign Clean Messages = yes

[Julian Field]

At 19:30 28/01/2002, you wrote:
>When I set "Sign Clean Messages = yes" no X-MailScanner header is added to
>scanned messages.
>It also looks like the Return-Path header disappears with that setting.
>When setting "Sign Clean Messages = no" the X-MailScanner and Return-Path
>headers are in place.
>Is this something in my setup? Or did I hit a bug in this great program?

You found a teensy bug. "Sign Clean Messages" disables the "X-MailScanner"
header on clean messages.
If you want to fix this now, and can't wait for the next version, here's a
patch:

*** sendmail.pl.3.04-1  Mon Jan 28 20:08:15 2002
--- sendmail.pl Mon Jan 28 20:08:50 2002
***************
*** 506,512 ****

       # Construct all the new headers
       $newheaders = MTA::ConstructHeaders($entities->{$id}->stringify_header);
!     $newheaders = AddInfectedHeader($newheaders) if $Clean ne 'clean';
       if (defined($IsSpam->{$id})) {
         $newheaders = MTA::AddHeader($newheaders, $Config::SpamHeader,
                                      $IsSpam->{$id});
--- 506,513 ----

       # Construct all the new headers
       $newheaders = MTA::ConstructHeaders($entities->{$id}->stringify_header);
!     $newheaders = AddCleanHeader($newheaders) if $Clean eq 'clean';
!     $newheaders = AddInfectedHeader($newheaders) if $Clean eq 'dirty';
       if (defined($IsSpam->{$id})) {
         $newheaders = MTA::AddHeader($newheaders, $Config::SpamHeader,
                                      $IsSpam->{$id});


--
Julian Field                Teaching Systems Manager
jkf at ecs.soton.ac.uk         Dept. of Electronics & Computer Science
Tel. 023 8059 2817          University of Southampton
                             Southampton SO17 1BJ