[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#628886: marked as done (PTS: strip whitespace from pre-redirection URLs)

Your message dated Fri, 3 Jun 2011 18:10:17 +0200
with message-id <20110603161016.GA20898@rivendell.home.ouaza.com>
and subject line Re: Bug#628886: PTS: strip whitespace from pre-redirection URLs
has caused the Debian Bug report #628886,
regarding PTS: strip whitespace from pre-redirection URLs
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
628886: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=628886
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: qa.debian.org
Severity: wishlist
User: debian-qa@lists.debian.org
Usertags: pts
Tags: patch

Please apply the below patch to the PTS apache configuration to strip
whitespace from input URLs before performing the redirects. This is
useful when copy and pasting package names and accidentally including
extra whitespace.

--- a/apache.conf
+++ b/apache.conf
@@ -15,11 +15,11 @@
 RewriteEngine on
 RewriteRule ^/$ /common/index.html [L,R]
 RewriteRule ^/favicon\.ico$ http://www.debian.org/favicon.ico [L,R]
-RewriteRule ^/lib([^/])([^/]+)$ /lib$1/lib$1$2.html [L,R]
-RewriteRule ^/([^/])([^/]+)$ /$1/$1$2.html [L,R]
-RewriteCond %{QUERY_STRING} ^src=lib(.)(.+)$
+RewriteRule ^/\s*lib([^/])([^/]+)\s*$ /lib$1/lib$1$2.html [L,R]
+RewriteRule ^/\s*([^/])([^/]+)\s*$ /$1/$1$2.html [L,R]
+RewriteCond %{QUERY_STRING} ^src=\s*lib(.)(.+)\s*$
 RewriteRule ^/common/index.html$ /lib%1/lib%1%2.html? [L,R,NE]
-RewriteCond %{QUERY_STRING} ^src=(.)(.+)$
+RewriteCond %{QUERY_STRING} ^src=\s*(.)(.+)\s*$
 RewriteRule ^/common/index.html$ /%1/%1%2.html? [L,R,NE]
 </VirtualHost>
 

-- 
bye,
pabs

http://wiki.debian.org/PaulWise

Attachment: signature.asc
Description: This is a digitally signed message part


--- End Message ---
--- Begin Message --- 
On Fri, 03 Jun 2011, Paul Wise wrote:
> Bah, had some typos, fixed:

Thanks, applied, will be live next time apache is reloaded.

That said usually "[^\s]" is written as "\S". I supposed it
would work here to but I kept the tested syntax you gave me.

Cheers,
-- 
Raphaël Hertzog ◈ Debian Developer

Follow my Debian News ▶ http://RaphaelHertzog.com (English)
                      ▶ http://RaphaelHertzog.fr (Français)

--- End Message ---
Reply to: