From f18dc26f65f6b31fb48a1c741cd0bc282376ddc4 Mon Sep 17 00:00:00 2001 From: Thomas Hochstein Date: Wed, 16 Jun 2010 21:37:43 +0200 Subject: [PATCH 1/1] Analyze failure codes, don't fail on temporary failures. Add analyze_smtp_reply(). Fixes #8. Signed-off-by: Thomas Hochstein --- checkmail.pl | 33 ++++++++++++++++++++++++++++----- 1 file changed, 28 insertions(+), 5 deletions(-) diff --git a/checkmail.pl b/checkmail.pl index e93a946..c7117b3 100644 --- a/checkmail.pl +++ b/checkmail.pl @@ -202,7 +202,7 @@ sub checksmtp { my ($success,$code,@message) = try_rcpt_to(\$smtp,$address,$logr); # connection failure? if ($success < 0) { - $status = connection_failed(); + $status = connection_failed(@message); # delivery attempt was successful? } elsif ($success) { # -r: try random address (which should be guaranteed to be invalid) @@ -211,7 +211,7 @@ sub checksmtp { my ($success,$code,@message) = try_rcpt_to(\$smtp,$config{'rand'}.'@'.$domain,$logr); # connection failure? if ($success < 0) { - $status = connection_failed(); + $status = connection_failed(@message); # verification impossible? } elsif ($success) { $status = 3; @@ -298,16 +298,18 @@ sub print_dns_result { # IN : \$smtp : a reference to an SMTP object # $recipient: a mail address # \$log : reference to the log (to be printed out via -l) -# OUT: $success: true or false +# OUT: $success: exit code (0 for false, 1 for true, -1 for tempfail) # $code : SMTP status code # $message: SMTP status message # \$log will be changed sub try_rcpt_to { my($smtpr,$recipient,$logr)=@_; $$logr .= sprintf("RCPT TO:<%s>\n",$recipient); - my $success = $$smtpr->to($recipient); + my $success; + $$smtpr->to($recipient); if ($$smtpr->code) { log_smtp_reply($logr,$$smtpr->code,$$smtpr->message); + $success = analyze_smtp_reply($$smtpr->code,$$smtpr->message); } else { $success = -1; $$logr .= "---Connection failure---\n"; @@ -328,11 +330,32 @@ sub log_smtp_reply { return; } +############################### analyze_smtp_reply ############################## +# analyze SMTP response codes and messages +# IN : $code : SMTP status code +# @message : SMTP status message +# OUT: exit code (0 for false, 1 for true, -1 for tempfail) +sub analyze_smtp_reply { + my($code,@message)=@_; + my $type = substr($code, 0, 1); + if ($type == 2) { + return 1; + } elsif ($type == 5) { + return 0; + } elsif ($type == 4) { + return -1; + }; + return -1; +} + ############################## connection_failed ############################### # print failure message and return status 1 +# IN : @message : SMTP status message # OUT: 1 sub connection_failed { - print " > Connection failure.\n" if !($options{'q'}); + my(@message)=@_; + print " ! Connection failed or other temporary failure.\n" if !($options{'q'}); + printf(" %s\n",join(' ',@message)) if @message; return 1; } -- 2.20.1