Patch #26522

repository routing bug when file path starts with (browse|entry|raw|changes|annotate|diff)/

Added by Jens Krämer 3 months ago. Updated 3 months ago.

Status:NewStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:SCM
Target version:4.0.0

Description

If the first component of a path in a repository equals one of the strings above, the wrong route gets selected in the case of links without repository_id (as they are generated from wiki source: links to the default repository of a project) and the intended action is mistaken for the repository_id parameter. Attached is a patch to the repository routing test illustrating the issue.

I would suggest to change repository#identifier_param to always return an identifier (see second patch) and remove the repository routes without repository_id (rows 280ff in current config/routes.rb).

repository_routing_test.patch Magnifier - illustrates failure of route for certain path (555 Bytes) Jens Krämer, 2017-07-25 11:42

repository_identifier_param.patch Magnifier - always return a repository identifier (381 Bytes) Jens Krämer, 2017-07-25 11:42

History

#1 Updated by Toshi MARUYAMA 3 months ago

  • Target version set to 4.0.0

Also available in: Atom PDF