Bug #100950 for Net-DNS-DynDNS: change request + possible bug

Tue Dec 16 12:07:04 2014 james [...] ziobro.info - Ticket created

Subject:	change request + possible bug
Date:	Tue, 16 Dec 2014 12:06:49 -0500
To:	bug-Net-DNS-DynDNS [...] rt.cpan.org
From:	Jim Ziobro <james [...] ziobro.info>

In DynDNS.pm the "if" statement at line 459 is modified to read: if ( ( $code eq 'good' ) || ( $code eq 'nochg' ) || $code eq '200') { # 200 is changeIP.com response see: # http://www.changeip.com/accounts/knowledgebase.php?action=displayarticle&id=47 if ($result) { In routine *_parse_content* the variable *$result* is declared and used but it does appear to be set anywhere. Do you want that library to support other DDNS providers? Ciao, //Z\\ Jim Ziobro

Wed Dec 17 05:13:47 2014 DDICK [...] cpan.org - Correspondence added

On Wed Dec 17 04:07:04 2014, james@ziobro.info wrote: Show quoted text

> In DynDNS.pm the "if" statement at line 459 is modified to read: > > if ( ( $code eq 'good' ) || ( $code eq 'nochg' ) > || $code eq '200') { # 200 is changeIP.com response see: > # > http://www.changeip.com/accounts/knowledgebase.php?action=displayarticle&id=47 > if ($result) { > > In routine *_parse_content* the variable *$result* is declared and > used > but it does appear to be set anywhere. > > Do you want that library to support other DDNS providers? > > Ciao, > //Z\\ > Jim Ziobro

Thinking about it. Looks like http://www.changeip.com/accounts/knowledgebase.php?action=displayarticle&id=34 is useful too.

Wed Dec 17 05:13:47 2014 The RT System itself - Status changed from 'new' to 'open'

Sat Aug 22 07:19:33 2015 DDICK [...] cpan.org - Correspondence added

On Wed Dec 17 21:13:47 2014, DDICK wrote: Show quoted text

> On Wed Dec 17 04:07:04 2014, james@ziobro.info wrote:

> > In DynDNS.pm the "if" statement at line 459 is modified to read: > > > > if ( ( $code eq 'good' ) || ( $code eq 'nochg' ) > > || $code eq '200') { # 200 is changeIP.com response see: > > # > > http://www.changeip.com/accounts/knowledgebase.php?action=displayarticle&id=47 > > if ($result) { > > > > In routine *_parse_content* the variable *$result* is declared and > > used > > but it does appear to be set anywhere. > > > > Do you want that library to support other DDNS providers? > > > > Ciao, > > //Z\\ > > Jim Ziobro

> > Thinking about it. Looks like > http://www.changeip.com/accounts/knowledgebase.php?action=displayarticle&id=34 > is useful too.

Okay. Took longer than i wanted to. Patch included. Send more if needed.

Sat Aug 22 07:19:38 2015 DDICK [...] cpan.org - Status changed from 'open' to 'resolved'