Skip Menu |

This queue is for tickets about the Pod-Perldoc CPAN distribution.

Report information
The Basics
Id: 107874
Status: patched
Priority: 0/
Queue: Pod-Perldoc
People
Owner: Nobody in particular
Requestors: NICZERO [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 3.25
Fixed in: (no value)

History Show all quoted text
  Tue Oct 20 05:17:37 2015 NICZERO [...] cpan.org - Ticket created
Subject: perldoc fails on https without non-core modules; resulting message not helpful
perldoc can serve pod from http-served .pm files just fine, and this is very good. But it's becoming increasingly popular for servers to redirect to https (metacpan, github, ...) and perldoc will fail with an unhelpful message No documentation found for "https://metacpan.org/source/.../DBIC.pm" The code could catch if the failure URL matches ^https, check if require IO::Socket::SSL or require Net::SSLeay fails, and append the error message with a suggestion to install the missing module. (Note that checking the original URL isn't enough.) thanks, Nic
  Tue Oct 20 10:50:04 2015 mrallen1 [...] yahoo.com - Correspondence added
On Tue Oct 20 05:17:37 2015, NICZERO wrote: Show quoted text
> The code could catch if the failure URL matches ^https, check if > require IO::Socket::SSL or require Net::SSLeay fails, and append the > error message with a suggestion to install the missing module. (Note > that checking the original URL isn't enough.)
Thanks for the suggestion - I'll take a look at including it in a future release.
  Tue Oct 20 10:50:04 2015 The RT System itself - Status changed from 'new' to 'open'
  Wed Mar 01 17:33:08 2017 mrallen1 [...] yahoo.com - Correspondence added
On Tue Oct 20 10:50:04 2015, mallen wrote: Show quoted text
> On Tue Oct 20 05:17:37 2015, NICZERO wrote:
> > The code could catch if the failure URL matches ^https, check if > > require IO::Socket::SSL or require Net::SSLeay fails, and append the > > error message with a suggestion to install the missing module. (Note > > that checking the original URL isn't enough.)
> > Thanks for the suggestion - I'll take a look at including it in a > future release.
I have a branch on github (rt107874) with hopefully a more helpful message. I plan to incorporate this patch in 3.28
  Wed Mar 01 17:33:14 2017 mrallen1 [...] yahoo.com - Status changed from 'open' to 'patched'