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

Bug#83640: web pages: CGIs broken wrt '+'



severity 72282 important
merge 83640 72282
thanks

On Fri, Jan 26, 2001 at 06:53:11PM -0500, James A. Treacy wrote:
> On Fri, Jan 26, 2001 at 03:26:29PM +0000, Yann Dirson wrote:
> > Package: www.debian.org
> > Severity: important
> > 
> > * http://packages.debian.org/g++ translates to
> > http://cgi.debian.org/cgi-bin/search_packages.pl?keywords=g++&searchon=names&version=all&release=all
> > 
> > Should read "keywords=g%2B%2B" instead of "keywords=g++"
> > 
> This is a known bug which I believe was even filed as a bug, but can't

OK, merging.

> be bothered to look up right now. The problem is with the rewrite rule,
> which no one has worked out how to fix.

OK, looks like involved rules are:

bugs.debian.org:

   RewriteRule ^/([^/]+$) /cgi-bin/pkgreport.cgi?pkg=$1 [L,R]

packages.debian.org:

   RewriteRule ^/([^/]+)$ http://cgi.debian.org/cgi-bin/search_packages.pl?keywords=$1\&searchon=names\&version=all\&release=all [L]

In both cases, $1 should be encoded while rewriting.  Maybe it should even
be decoded first (which seems to be already done), so that bugs.d.o/g%2B%2B
gives the expected pkg=g%2B%2B

A solution, after reading doc/apache/manual/misc/rewriteguide.html, is given
by the last example - using RewriteMap.


Probably pkgreport.cgi should decode the package name in the packages.d.o
link.  This should be trivial, but will only be really useful when the above
is fixed.

No idea a-priori about why libg++2.8.3.1 misses its links.

-- 
Yann Dirson    <ydirson@altern.org> |    Why make M$-Bill richer & richer ?
debian-email:   <dirson@debian.org> |   Support Debian GNU/Linux:
                                    | Cheaper, more Powerful, more Stable !
http://ydirson.free.fr/             | Check <http://www.debian.org/>



Reply to: