We don't check to see if the error we are substituting '250 OK' for is 
actually a 5xx error. This causes an issue with the '451 Connection timeout, 
try later' message on connection timeout. I don't think we really want to 
send 250 OK instead of the 451 message.

Solution below:

sub error {
  my($fh,$l)[EMAIL PROTECTED];
  d(29);
  my $this=$Con{$fh};
  if( $l=~/^\.[\r\n]*$/ || defined($this->{bdata}) && $this->{bdata}<=0) {

  $this->{error} = "250 OK" if $this->{error} =~ /^5[0-9][0-9]/
   && ( $send250OK || ( $this->{ispip} && $send250OKISP ) );

 mlog( $fh, "[SMTP Error] $this->{error}", 1, 1 ) if $replyLogging && 
$this->{error}!~/250 OK/;
    print $fh $this->{error}."\r\n";
    done2($fh);
  }
}



-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2008.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
_______________________________________________
Assp-test mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/assp-test

Reply via email to