Bug 970283: Bump URI version to 1.55 to correctly escape single quotes in URLs

r=dkl a=justdave

diff --git a/Bugzilla/Bug.pm b/Bugzilla/Bug.pm
index a96731949c034763d2fedd64ef7df4e8cb869299..128d2a4cd620d2ba6a5b9af38a4d55553281d35c 100644 (file)
--- a/Bugzilla/Bug.pm
+++ b/Bugzilla/Bug.pm
@@ -32,8 +32,6 @@ use Bugzilla::BugUrl;
 use List::MoreUtils qw(firstidx uniq part);
 use List::Util qw(min max first);
 use Storable qw(dclone);
-use URI;
-use URI::QueryParam;
 use Scalar::Util qw(blessed);
 
 use parent qw(Bugzilla::Object Exporter);

diff --git a/Bugzilla/BugUrl.pm b/Bugzilla/BugUrl.pm
index df72d9cf78856b6d7b99a093ec65e279ddb0590a..39b985d290d7aa4baed2809728e9bbe077df65ee 100644 (file)
--- a/Bugzilla/BugUrl.pm
+++ b/Bugzilla/BugUrl.pm
@@ -17,6 +17,7 @@ use Bugzilla::Error;
 use Bugzilla::Constants;
 use Bugzilla::Hook;
 
+use URI;
 use URI::QueryParam;
 
 ###############################

diff --git a/Bugzilla/Install/Requirements.pm b/Bugzilla/Install/Requirements.pm
index e7212915318e1f645169bcb074215da0e058ad02..74cbd82b472c91c742556390338cdce59622013f 100644 (file)
--- a/Bugzilla/Install/Requirements.pm
+++ b/Bugzilla/Install/Requirements.pm
@@ -148,9 +148,8 @@ sub REQUIRED_MODULES {
     {
         package => 'URI',
         module  => 'URI',
-        # This version properly handles a semicolon as the delimiter
-        # in a URL query string.
-        version => '1.37',
+        # Follows RFC 3986 to escape characters in URI::Escape.
+        version => '1.55',
     },
     # 0.32 fixes several memory leaks in the XS version of some functions.
     {
@@ -396,12 +395,6 @@ sub OPTIONAL_MODULES {
     },
 
     # memcached
-    {
-        package => 'URI-Escape',
-        module  => 'URI::Escape',
-        version => 0,
-        feature => ['memcached'],
-    },
     {
         package => 'Cache-Memcached',
         module  => 'Cache::Memcached',