On Wed, Nov 20, 2019 at 07:29:10PM +0100, Giovanni Bechis wrote:
> anybody against this diff ?
>
> Giovanni
>
> Index: lib/Mail/SpamAssassin/DnsResolver.pm
> ===================================================================
> --- lib/Mail/SpamAssassin/DnsResolver.pm (revision 1870052)
> +++ lib/Mail/SpamAssassin/DnsResolver.pm (working copy)
> @@ -858,7 +858,7 @@
> if ($rcode eq 'REFUSED' || $id =~ m{^\d+/NO_QUESTION_IN_PACKET\z})
> {
> # the failure was already reported above
> } else {
> - info("dns: no callback for id %s, ignored; packet: %s",
> + dbg("dns: no callback for id %s, ignored; packet: %s",
> $id, $packet ? $packet->string : "undef" );
> }
> # report a likely matching query for diagnostic purposes
If you would check trunk it's already there. Please use the same code for
uniformity.
info("dns: no callback for id $id, ignored, packet on next debug
line");
# prevent filling normal logs with huge packet dumps
dbg("dns: %s", $packet ? $packet->string : "undef");
