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
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
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