Skip to content
Browse files

temp_resolver_failed is a transaction note

 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.
  • Loading branch information...
1 parent 24d09fa commit b7668c046879ab3e55a15bfec7d2708680cec58e Peter J. Holzer committed with rspier
Showing with 2 additions and 2 deletions.
  1. +2 −2 plugins/require_resolvable_fromhost
View
4 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
+# vim: ts=2 sw=2 expandtab syn=perl

0 comments on commit b7668c0

Please sign in to comment.
Something went wrong with that request. Please try again.