Skip Menu |
 

This queue is for tickets about the URI CPAN distribution.

Report information
The Basics
Id: 117066
Status: new
Priority: 0/
Queue: URI

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

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



Subject: t/storable-test.pl coerces Storable to runtime load modules
Download (untitled) / with headers
text/plain 755b
We patch Storable to NOT runtime load modules when it thaws something. This is an old back and forth with p5p where they basically stated that it's not safe to use Storable for this sorta thing (yes it's unsafe. don't do it) As a result, t/storable.t is failing on our perl. This patch would make the test pass for us if you are interested in taking it. If not feel free to reject: diff --git a/modules/URI/URI/t/storable-test.pl b/modules/URI/URI/t/storable-test.pl index 33deb6f..51ce137 100644 --- a/modules/URI/URI/t/storable-test.pl +++ b/modules/URI/URI/t/storable-test.pl @@ -2,6 +2,9 @@ use strict; use warnings; use Storable; +use URI::http (); +use URI (); + if (@ARGV && $ARGV[0] eq "store") { require URI; require URI::URL;


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.