The mail hook sets a transaction note 'temp_resolver_failed', but the rcpt hook queried a connection note of the same name (which didn't exist, of course). Now it queries the transaction note.
--- plugins/require_resolvable_fromhost | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/require_resolvable_fromhost b/plugins/require_resolvable_fromhost index e9e2a91..cdb1890 100644 --- a/plugins/require_resolvable_fromhost +++ b/plugins/require_resolvable_fromhost @@ -41,7 +41,7 @@ sub hook_mail { sub hook_rcpt { my ($self, $transaction, $recipient, %args) = @_; - if (my $host = $self->qp->connection->notes('temp_resolver_failed')) { + if (my $host = $transaction->notes('temp_resolver_failed')) { # default of temp_resolver_failed is DENYSOFT return Qpsmtpd::DSN->temp_resolver_failed("Could not resolve " . $host); } @@ -148,4 +148,4 @@ sub mx_valid { return 0; } -# vim: ts=2 sw=2 expandtab syn=perl \ No newline at end of file +# vim: ts=2 sw=2 expandtab syn=perl -- 1.7.2.3 -- _ | Peter J. Holzer | Web 2.0 könnte man also auch übersetzen als |_|_) | Sysadmin WSR | "Netz der kleinen Geister". | | | h...@hjp.at | __/ | http://www.hjp.at/ | -- Oliver Cromm in desd
signature.asc
Description: Digital signature