Skip Menu |

Preferred bug tracker

Please email the preferred bug tracker to report your issue.

This queue is for tickets about the Geo-WebService-Elevation-USGS CPAN distribution.

Report information
The Basics
Id: 127252
Status: resolved
Priority: 0/
Queue: Geo-WebService-Elevation-USGS
People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc: wyant [...] cpan.org
AdminCc:
Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: 0.110

History Show all quoted text
  Sat Sep 29 03:43:26 2018 SREZIC [...] cpan.org - Ticket created
CC: wyant [...] cpan.org
Subject: Test suite started to fail
The t/elev.t tests started to fail: ... # Failed test 'getElevation succeeded' # at t/elev.t line 65. # malformed JSON string, neither array, object, number, string or atom, at character offset 2 (before "<!DOCTYPE html PUBLI...") at /home/cpansand/.cpan/build/2018092908/Geo-WebService-Elevation-USGS-0.109-BrfZX5/blib/lib/Geo/WebService/Elevation/USGS.pm line 724. # at t/elev.t line 62. ... (etc) ... There are some recent passes on CPAN Testers, but this happens only if the elev.t is skipped (like in the the report http://www.cpantesters.org/cpan/report/4b09a552-c3b0-11e8-86e3-404589cf513b ) --- I guess the skip happens only because the website redirects to https and LWP::Protocol::https is not yet installed (so that module should be listed in the prereqs).
  Sat Sep 29 18:22:56 2018 wyant [...] cpan.org - Correspondence added
Thank you very much for your report. What seems to have happened is that the API disappeared from the URL I had programmed against. Fortunately Google was able to figure out where it went, because I was unable to after about 15 minutes' flailing around on the U. S. Geological Service's web sites. I hate to put out a production release without letting CPAN testers chew on it a bit, but seemed to be an exception to the rule. Unfortunately I did all this before reading my mail (and thus your RT ticket), so I was unable to think about LWP::Protocol::https before release. Fortunately the new URL is http:, and $ env PERL5OPT=-MDevel::Hide=LWP::Protocol::https ./Build test --verbose seems not only to succeed but not to skip anything. I have to start doing the author test (or _an_ author test anyhow) with LWP::Protocol::https hidden so I will know when it needs to become a prerequisite. Of course, none of this addresses the meta-problem that, if you automate access to a web service, you no longer visit the web page to see any notices that the service is being moved.
  Sat Sep 29 18:22:56 2018 The RT System itself - Status changed from 'new' to 'open'
  Sat Sep 29 18:23:22 2018 wyant [...] cpan.org - Status changed from 'open' to 'patched'
  Sat Sep 29 18:23:22 2018 wyant [...] cpan.org - Severity Critical added
  Sun Sep 30 10:09:21 2018 SREZIC [...] cpan.org - Correspondence added
RT-Send-CC: wyant [...] cpan.org
On 2018-09-29 18:22:56, WYANT wrote: Show quoted text
> Thank you very much for your report. > > What seems to have happened is that the API disappeared from the URL I > had programmed against. Fortunately Google was able to figure out > where it went, because I was unable to after about 15 minutes' > flailing around on the U. S. Geological Service's web sites. > > I hate to put out a production release without letting CPAN testers > chew on it a bit, but seemed to be an exception to the rule. > > Unfortunately I did all this before reading my mail (and thus your RT > ticket), so I was unable to think about LWP::Protocol::https before > release. Fortunately the new URL is http:, and > > $ env PERL5OPT=-MDevel::Hide=LWP::Protocol::https ./Build test > --verbose > > seems not only to succeed but not to skip anything. I have to start > doing the author test (or _an_ author test anyhow) with > LWP::Protocol::https hidden so I will know when it needs to become a > prerequisite. > > Of course, none of this addresses the meta-problem that, if you > automate access to a web service, you no longer visit the web page to > see any notices that the service is being moved.
CPAN Testers results look good now: http://matrix.cpantesters.org/?dist=Geo-WebService-Elevation-USGS+0.110 So I am closing this issue.
  Sun Sep 30 10:09:22 2018 SREZIC [...] cpan.org - Status changed from 'patched' to 'resolved'
  Sun Sep 30 10:09:22 2018 SREZIC [...] cpan.org - Fixed in 0.110 added