byterazor/qpsmtpd
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't interpret invalid return codes as OK

Browse Source
This commit is contained in:
Jared Johnson 2015-02-03 17:01:52 -06:00
parent b24f766cc5
commit dcdda3a5e7
2 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/Qpsmtpd.pm
View File

@ -302,6 +302,10 @@ sub run_continuation {
$self->log(LOGERROR, $log_msg . "undef!"); $self->log(LOGERROR, $log_msg . "undef!");
next; next;
} }
if ( !return_code($r[0]) ) {
$self->log(LOGERROR, $log_msg . $r[0]);
next;
}
if ($tran) { if ($tran) {
my $tnotes = $tran->notes($name); my $tnotes = $tran->notes($name);

13
t/qpsmtpd.t
View File

@ -149,12 +149,13 @@ sub __run_continuation {
descr => 'DECLINED -> DENY', descr => 'DECLINED -> DENY',
}, },
# TODO: ignore invalid return codes rather than treating them like OK # TODO: ignore invalid return codes rather than treating them like OK
#{ {
# hooks => [ [123456,undef], [DENY, 'goaway'] ], hooks => [ [123456,undef], [DENY, 'goaway'] ],
# expected_response => '550/goaway', expected_response => '550/goaway',
# disconnected => 0, disconnected => 0,
# descr => 'INVALID -> DENY', logged => 'LOGERROR:Plugin ___FakeHook___, hook helo returned 123456',
#}, descr => 'INVALID -> DENY',
},
{ {
hooks => [ sub { die "dead\n" }, [DENY, 'begone'] ], hooks => [ sub { die "dead\n" }, [DENY, 'begone'] ],
expected_response => '550/begone', expected_response => '550/begone',