Skip Menu |
 

This queue is for tickets about the Net-DNS CPAN distribution.

Report information
The Basics
Id: 95859
Status: rejected
Priority: 0/
Queue: Net-DNS

People
Owner: Nobody in particular
Requestors: ppisar [...] redhat.com
rick [...] clbcm.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.75
Fixed in: (no value)



Subject: Net::DNS::Resolver::Programmable's NXDOMAIN reply is interpreted as an error
Download (untitled) / with headers
text/plain 708b
While debugging Mail-SPF failure <https://rt.cpan.org/Public/Bug/Display.html?id=95858> I come to a regression in Net-DNS between 0.74 and 0.75. This code: #!/usr/bin/perl use strict; use warnings; use Net::DNS::Resolver::Programmable; my $resolver = Net::DNS::Resolver::Programmable->new( resolver_code => sub { return ('NXDOMAIN', undef) } ); my $packet = $resolver->send('example.com', 'A', 'IN'); use Data::Dumper; print Dumper($packet); print $packet->header->rcode, "\n" if defined $packet; run with Net-DNS-0.74 returns defined $packet and $packet->header->rcode is 'NXDOMAIN', while run with Net-DNS-0.75 it returns undef $packet. This was regression was introduced in SVN revision 1171.
Download (untitled) / with headers
text/plain 941b
I have created a ticket in the Net-DNS-Resolver-Programmable queue for this issue: https://rt.cpan.org/Ticket/Display.html?id=95901 On Thu 22 Mei 2014 07:01:18, ppisar wrote: Show quoted text
> While debugging Mail-SPF failure > <https://rt.cpan.org/Public/Bug/Display.html?id=95858> I come to a > regression in Net-DNS between 0.74 and 0.75. > > This code: > > #!/usr/bin/perl > use strict; > use warnings; > use Net::DNS::Resolver::Programmable; > > my $resolver = Net::DNS::Resolver::Programmable->new( > resolver_code => sub { return ('NXDOMAIN', undef) } > ); > my $packet = $resolver->send('example.com', 'A', 'IN'); > > use Data::Dumper; > print Dumper($packet); > print $packet->header->rcode, "\n" if defined $packet; > > > run with Net-DNS-0.74 returns defined $packet and $packet->header-
> >rcode is 'NXDOMAIN', while run with Net-DNS-0.75 it returns undef
> $packet. > > This was regression was introduced in SVN revision 1171.
Download (untitled) / with headers
text/plain 1.3k
Hi Petr, Since it does not affects Mail-SPF operations but just the unit tests, and is caused by use of Net::DNS internals by Net::DNS::Resolver::Programmable (which is not us), I dare to say the urgency for us to put the internal back is not that big... Instead I asked Net-DNS-Resolver-Programmable to fix the issue on their side. Do you agree? -- Willem On Fri 23 Mei 2014 17:31:16, NLNETLABS wrote: Show quoted text
> I have created a ticket in the Net-DNS-Resolver-Programmable queue for > this issue: > https://rt.cpan.org/Ticket/Display.html?id=95901 > > On Thu 22 Mei 2014 07:01:18, ppisar wrote:
> > While debugging Mail-SPF failure > > <https://rt.cpan.org/Public/Bug/Display.html?id=95858> I come to a > > regression in Net-DNS between 0.74 and 0.75. > > > > This code: > > > > #!/usr/bin/perl > > use strict; > > use warnings; > > use Net::DNS::Resolver::Programmable; > > > > my $resolver = Net::DNS::Resolver::Programmable->new( > > resolver_code => sub { return ('NXDOMAIN', undef) } > > ); > > my $packet = $resolver->send('example.com', 'A', 'IN'); > > > > use Data::Dumper; > > print Dumper($packet); > > print $packet->header->rcode, "\n" if defined $packet; > > > > > > run with Net-DNS-0.74 returns defined $packet and $packet->header-
> > > rcode is 'NXDOMAIN', while run with Net-DNS-0.75 it returns undef
> > $packet. > > > > This was regression was introduced in SVN revision 1171.
Subject: Re: [rt.cpan.org #95859] Net::DNS::Resolver::Programmable's NXDOMAIN reply is interpreted as an error
Date: Fri, 23 May 2014 15:51:43 -0600
To: bug-Net-DNS [...] rt.cpan.org
From: Rob Brown <bbb [...] cpan.org>
Download (untitled) / with headers
text/plain 1.9k
If Net::DNS::Resolver::Programmable cannot avoid using internals of Net::DNS, then the hope is that Programmable.pm can be modified to handle both the old Net::DNS internal way and the new way identically without too much of a performance penalty. Or simply alter it to only work with the newer Net::DNS 0.75 version. On Fri, May 23, 2014 at 3:36 PM, NLnet Labs via RT <bug-Net-DNS@rt.cpan.org>wrote: Show quoted text
> Queue: Net-DNS > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=95859 > > > Hi Petr, > > Since it does not affects Mail-SPF operations but just the unit tests, and > is caused by use of Net::DNS internals by Net::DNS::Resolver::Programmable > (which is not us), I dare to say the urgency for us to put the internal > back is not that big... Instead I asked Net-DNS-Resolver-Programmable to > fix the issue on their side. > > Do you agree? > > -- Willem > > On Fri 23 Mei 2014 17:31:16, NLNETLABS wrote:
> > I have created a ticket in the Net-DNS-Resolver-Programmable queue for > > this issue: > > https://rt.cpan.org/Ticket/Display.html?id=95901 > > > > On Thu 22 Mei 2014 07:01:18, ppisar wrote:
> > > While debugging Mail-SPF failure > > > <https://rt.cpan.org/Public/Bug/Display.html?id=95858> I come to a > > > regression in Net-DNS between 0.74 and 0.75. > > > > > > This code: > > > > > > #!/usr/bin/perl > > > use strict; > > > use warnings; > > > use Net::DNS::Resolver::Programmable; > > > > > > my $resolver = Net::DNS::Resolver::Programmable->new( > > > resolver_code => sub { return ('NXDOMAIN', undef) } > > > ); > > > my $packet = $resolver->send('example.com', 'A', 'IN'); > > > > > > use Data::Dumper; > > > print Dumper($packet); > > > print $packet->header->rcode, "\n" if defined $packet; > > > > > > > > > run with Net-DNS-0.74 returns defined $packet and $packet->header-
> > > > rcode is 'NXDOMAIN', while run with Net-DNS-0.75 it returns undef
> > > $packet. > > > > > > This was regression was introduced in SVN revision 1171.
> > > >
Subject: Re: [rt.cpan.org #95859] Net::DNS::Resolver::Programmable's NXDOMAIN reply is interpreted as an error
Date: Sat, 24 May 2014 00:05:04 +0200
To: bug-Net-DNS [...] rt.cpan.org
From: Willem Toorop <willem [...] nlnetlabs.nl>
Download (untitled) / with headers
text/plain 2.5k
I think it could be avoidable but I've given them a patch that tests old internal way, and uses the new way otherwise (see https://rt.cpan.org/Ticket/Display.html?id=95901). The new way is in place since version 0.69 b.t.w. so it could depend on that minimum version. But I believe it can avoid using the internal altogether as well by simply trying to set the result code and catching the error ... (but have not provided a patch for that yet) op 23-05-14 23:51, Rob Brown via RT schreef: Show quoted text
> Queue: Net-DNS > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=95859 > > > If Net::DNS::Resolver::Programmable cannot avoid using internals of > Net::DNS, then the hope is that Programmable.pm can be modified to handle > both the old Net::DNS internal way and the new way identically without too > much of a performance penalty. Or simply alter it to only work with the > newer Net::DNS 0.75 version. > > > > > On Fri, May 23, 2014 at 3:36 PM, NLnet Labs via RT > <bug-Net-DNS@rt.cpan.org>wrote: >
>> Queue: Net-DNS >> Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=95859 > >> >> Hi Petr, >> >> Since it does not affects Mail-SPF operations but just the unit tests, and >> is caused by use of Net::DNS internals by Net::DNS::Resolver::Programmable >> (which is not us), I dare to say the urgency for us to put the internal >> back is not that big... Instead I asked Net-DNS-Resolver-Programmable to >> fix the issue on their side. >> >> Do you agree? >> >> -- Willem >> >> On Fri 23 Mei 2014 17:31:16, NLNETLABS wrote:
>>> I have created a ticket in the Net-DNS-Resolver-Programmable queue for >>> this issue: >>> https://rt.cpan.org/Ticket/Display.html?id=95901 >>> >>> On Thu 22 Mei 2014 07:01:18, ppisar wrote:
>>>> While debugging Mail-SPF failure >>>> <https://rt.cpan.org/Public/Bug/Display.html?id=95858> I come to a >>>> regression in Net-DNS between 0.74 and 0.75. >>>> >>>> This code: >>>> >>>> #!/usr/bin/perl >>>> use strict; >>>> use warnings; >>>> use Net::DNS::Resolver::Programmable; >>>> >>>> my $resolver = Net::DNS::Resolver::Programmable->new( >>>> resolver_code => sub { return ('NXDOMAIN', undef) } >>>> ); >>>> my $packet = $resolver->send('example.com', 'A', 'IN'); >>>> >>>> use Data::Dumper; >>>> print Dumper($packet); >>>> print $packet->header->rcode, "\n" if defined $packet; >>>> >>>> >>>> run with Net-DNS-0.74 returns defined $packet and $packet->header-
>>>>> rcode is 'NXDOMAIN', while run with Net-DNS-0.75 it returns undef
>>>> $packet. >>>> >>>> This was regression was introduced in SVN revision 1171.
>> >> >> >>
Subject: Re: [rt.cpan.org #95859] Net::DNS::Resolver::Programmable's NXDOMAIN reply is interpreted as an error
Date: Mon, 26 May 2014 10:09:27 +0200
To: NLnet Labs via RT <bug-Net-DNS [...] rt.cpan.org>
From: Petr Pisar <ppisar [...] redhat.com>
Download (untitled) / with headers
text/plain 693b
On Fri, May 23, 2014 at 05:36:07PM -0400, NLnet Labs via RT wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=95859 > > > Since it does not affects Mail-SPF operations but just the unit tests, and > is caused by use of Net::DNS internals by Net::DNS::Resolver::Programmable > (which is not us), I dare to say the urgency for us to put the internal back > is not that big... Instead I asked Net-DNS-Resolver-Programmable to fix the > issue on their side. > > Do you agree? >
Ok. I did not notice Net::DNS::Resolver::Programmable is a separate distribution. This is issue is not urgent for be. It just causes some smoke <http://matrix.cpantesters.org/?dist=Mail-SPF+v2.9.0>. -- Petr
Download (untitled)
application/pgp-signature 230b

Message body not shown because it is not plain text.

Covered in other tickets
Subject: Net::DNS Mail::SPF
Date: Tue, 30 Sep 2014 15:52:13 -0400
To: bug-Net-DNS [...] rt.cpan.org
From: Rick Labs <rick [...] clbcm.com>
Download (untitled) / with headers
text/plain 143k

Message body is not shown because it is too large.

Download (untitled) / with headers
text/html 164.9k

Message body is not shown because it is too large.

Subject: Net::DNS Mail::SPF
Date: Tue, 30 Sep 2014 15:52:13 -0400
To: bug-Net-DNS [...] rt.cpan.org
From: Rick Labs <rick [...] clbcm.com>
Download (untitled) / with headers
text/plain 143k

Message body is not shown because it is too large.

Download (untitled) / with headers
text/html 164.9k

Message body is not shown because it is too large.

Subject: Net::DNS Mail::SPF
Date: Tue, 30 Sep 2014 15:52:13 -0400
To: bug-Net-DNS [...] rt.cpan.org
From: Rick Labs <rick [...] clbcm.com>
Download (untitled) / with headers
text/plain 143k

Message body is not shown because it is too large.

Download (untitled) / with headers
text/html 164.9k

Message body is not shown because it is too large.

Download (untitled) / with headers
text/plain 149k

Message body is not shown because it is too large.



This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.