Noah Kantrowitz wrote:
I thought about adding this as a core system, but there are places where a
trailing / is probably legal. The biggest one is attachment URLs, since / is
a legal filename character on some OSes. Granted it isn't any major ones, so
probably it would be fine to add this at the level of the URL dispatcher.

The attachment module treats e.g. "/attachment/wiki/Projects/Trac/" and "/attachment/wiki/Projects/Trac" differently. The former shows the list of attachments for the Projects/Trac page, and the second shows the attachment with the name "Trac" attached to the page Projects.

Maybe the file name and the path should be reversed:

  /attachment/wiki/MyFile/Projects/Trac

would open the attachment named "MyFile" for page "Projects/Trac".

-- Remy

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to