FS#11117 - Django url matching doesn't handle special characters

Attached to Project: Arch Linux
Opened by Henning Garus (garns) - Monday, 04 August 2008, 18:55 GMT
Last edited by Dusty Phillips (Dusty) - Monday, 04 August 2008, 19:16 GMT
Task Type Bug Report
Category Web Sites
Status Closed
Assigned To Dusty Phillips (Dusty)
Architecture All
Severity Medium
Priority Normal
Reported Version None
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:

Packagenames are not strictly alphanumeric, a lot of packages hava a + in their name. There are also some pkgs with dots (.) in their name. There might be other special characters in pkgnames as well, however a quick search did not turn up any.

The url matching in Django does not handle these special characters, thus http://www.archlinux.org/packages/extra/i686/memtest86+/ and similar lead to a 404.


Steps to reproduce:

http://www.archlinux.org/packages/extra/i686/memtest86+/
http://www.archlinux.org/packages/extra/x86_64/wxgtk-2.6/
http://www.archlinux.org/packages/search/kdebase-runtime


I've attached a patch to correct this behaviour, I kept the matching restricted to a few special characters, some more could be added, if seen as necessary (shard for example).
   0001-Change-urls.py-to-handle... (1.7 KiB)
This task depends upon

Closed by  Dusty Phillips (Dusty)
Monday, 04 August 2008, 19:16 GMT
Reason for closing:  Fixed
Additional comments about closing:  I love people who submit patches =)
Comment by Greg (dolby) - Monday, 04 August 2008, 19:01 GMT
http://archlinux.org/pipermail/arch-general/2008-August/018307.html

Loading...