Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 69283
Status: resolved
Priority: 0/
Queue: Parse-DNS-Zone

People
Owner: Nobody in particular
Requestors: Hagedorn [...] uni-koeln.de
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 0.41
Fixed in: (no value)



Subject: Test for TTL is broken
in _parse_zone there is a test intended to ensure that each RR has a valid TTL. It looks like this: if(not defined $def_ttl) { carp("no ttl is set"); next; } That doesn't do what it's supposed to because it always fails if there is no default $TTL directive in the zone. We use generated zone files that have the TTL explicitly set in every line. Those zone files can't be parsed using 0.41. I think the code should be changed to this instead: if(not defined $ttl ) { carp("no ttl is set"); next; } That should work because a few lines earlier there's this statement: $ttl = defined $ttl ? $ttl : $def_ttl; So $ttl will only be undefined if $def_ttl is undefined.
Subject: Re: [rt.cpan.org #69283] Test for TTL is broken
Date: Tue, 5 Jul 2011 16:01:42 +0200
To: "Hagedorn [...] uni-koeln.de via RT" <bug-Parse-DNS-Zone [...] rt.cpan.org>
From: Olof Johansson <olof [...] ethup.se>
Hum, yes, you are quite correct. I've just moved, so I don't think I will be able to release a fix in the next couple of days, but probably this weekend. Will also update test case. Thank you, -- --------------------------------------------------------------- | Olof Johansson | | http://stdlib.se/ | | {mail,xmpp}: olof@ethup.se | | zibri on Freenode/OFTC/etc | ---------------------------------------------------------------
Download signature.asc
application/pgp-signature 198b

Message body not shown because it is not plain text.

I just released 0.42, it should fix this issue. It can take a while before it propagates to your CPAN mirror and search.cpan.org. Once again, thanks for the report. //Olof