Skip Menu |

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

Report information
The Basics
Id: 100994
Status: resolved
Priority: 0/
Queue: Net-DNS-DynDNS

People
Owner: Nobody in particular
Requestors: james [...] ziobro.info
Cc:
AdminCc:

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



Subject: A possible minor bug but really important for non-Dyn.COM users
Date: Thu, 18 Dec 2014 11:34:03 -0500
To: bug-Net-DNS-DynDNS [...] rt.cpan.org
From: Jim Ziobro <james [...] ziobro.info>
Other DDNS providers give long message in their status message thus we need to see whole message in error handling. Suggest using $content instead of $code in error message. I.e.: *************** *** 399,405 **** 'dnserr' => 'DNS error encountered', '911' => 'There is a problem or scheduled maintenance on our side', ); ! Carp::croak( $errors{$code} || "Unknown error:$content" ); } sub update { --- 399,405 ---- 'dnserr' => 'DNS error encountered', '911' => 'There is a problem or scheduled maintenance on our side', ); ! Carp::croak( $errors{$code} || "Unknown error:$code" ); } sub update {
On Fri Dec 19 03:34:17 2014, james@ziobro.info wrote: Show quoted text
> Other DDNS providers give long message in their status message thus we > need to see whole message in error handling. Suggest using $content > instead of $code in error message. I.e.: > *************** > *** 399,405 **** > 'dnserr' => 'DNS error encountered', > '911' => 'There is a problem or scheduled maintenance on > our side', > ); > ! Carp::croak( $errors{$code} || "Unknown error:$content" ); > } > > sub update { > --- 399,405 ---- > 'dnserr' => 'DNS error encountered', > '911' => 'There is a problem or scheduled maintenance on > our side', > ); > ! Carp::croak( $errors{$code} || "Unknown error:$code" ); > } > > sub update { >
Done. Thanks for the patch