Skip Menu |
 

This queue is for tickets about the CPAN CPAN distribution.

Report information
The Basics
Id: 82106
Status: open
Priority: 0/
Queue: CPAN

People
Owner: Nobody in particular
Requestors: SREZIC [...] cpan.org
Cc:
AdminCc:

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



Subject: lynx should not be used for downloading
Download (untitled) / with headers
text/plain 496b
The problem with lynx is that an error cannot be detected. So the following command lynx -source http://www.cpan.org/authors/id/S/SR/SREZIC/patches/thisfiledoesnotexist; echo $? shows exit code 0 (success) and would happily store the 404 error page in the local cache directory. Especially with patches there's no extra safety net by checking against the CHECKSUMS file. I think it's best to not use lynx at all --- nowadays LWP, Net::FTP, wget, or curl do a better job. Regards, Slaven
Download (untitled) / with headers
text/plain 748b
On 2012-12-19 11:26:25, SREZIC wrote: Show quoted text
> The problem with lynx is that an error cannot be detected. So the > following command > > lynx -source > http://www.cpan.org/authors/id/S/SR/SREZIC/patches/thisfiledoesnotexist; > echo $? > > shows exit code 0 (success) and would happily store the 404 error page > in the local cache directory. Especially with patches there's no extra > safety net by checking against the CHECKSUMS file. > > I think it's best to not use lynx at all --- nowadays LWP, Net::FTP, > wget, or curl do a better job.
Alternatively, when patches are downloaded then the CHECKSUMS checking could be done (currently it seems that this is not happening). In this case there would be no harm if lynx is used. Regards, Slaven


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.