ANNOUNCE: 4.01-5( sender notice problem)

[Rose, Bobby]

Julian, this problem is because of the uncommented line

next unless $message->{infected};

in the warnsenders subroutine to fix the Still Deliver Silent Viruses /
Deliver Cleaned Messages issue.  If I comment it out then sender notices
work again but if Deliver Silent Viruses = no and Deliver Cleaned
Messages = no then sender notices go out for Klez and Bugbear again
also.

-----Original Message-----
From: Rose, Bobby 
Sent: Tuesday, October 22, 2002 1:26 AM
To: MAILSCANNER at JISCMAIL.AC.UK
Subject: Re: ANNOUNCE: 4.01-5( postmaster notice problem)


Well this broke by this change between -3 and -5 in the MessageBatch.pm

@@ -544,7 +548,7 @@
   # Create all the email messages
   $counter = 0;
   while(($id, $message) = each %{$this->{messages}}) {
-    next unless $message->{infected};
+    next if !$message->{infected} || $message->{deleted};
     next unless MailScanner::Config::Value('sendnotices', $message);
     $postie = MailScanner::Config::Value('noticerecipient', $message);
     $headers{$postie} = $message->CreatePostmasterHeaders()

-----Original Message-----
From: Rose, Bobby 
Sent: Monday, October 21, 2002 11:10 PM
To: MAILSCANNER at JISCMAIL.AC.UK
Subject: Re: ANNOUNCE: 4.01-5 released


Actually sending the clean virus message to recipients is working here.
Just not notification to sender 4.01-3 rev and up and postmaster
notification since 4.01-5

This is on Solaris.



-----Original Message-----
From: Ben C. O. Grimm [mailto:mailscanner-sub at WIREHUB.NET] 
Sent: Monday, October 21, 2002 9:32 PM
To: MAILSCANNER at JISCMAIL.AC.UK
Subject: Re: ANNOUNCE: 4.01-5 released


On 22 Oct 2002 02:43:32 +0200, "Rose, Bobby" <brose at MED.WAYNE.EDU>
wrote:

> Ugh...  Besides not getting Warning messages sent, now I'm not getting

> postmaster notices after upgrading to this version.  I'm even trying 
> using the distro'd mailscanner.conf.

Same here: no warnings are sent at all (not to sender, not to
postmaster; not sending to intended recipients anyway, but that probably
does not work either ..).