Defect #7505

Subversion: Repository URL with space not handled correctly

Added by Adam Piotr Żochowski over 1 year ago. Updated about 1 year ago.

Status:New Start date:2011-01-31
Priority:Normal Due date:
Assignee:- % Done:

0%

Category:SCM
Target version:-
Affected version: Resolution:

Description

To avoid issues with any special characters in URL I use Tortoise svn and it's " copy url link " option while browing repository. So Redmine gets nice clean proper URLs:

https://server/projects/Project%20Name

However, new Redmine now is munging clean URLs. For example the above URL gets parsed as:

https://server/projects/Project%2520Name

as far as I can guess, the URL is meant to be escaped, however lib\redmine\scm\adapters\subversion_adapter.rb is doing too much of replacing :

        def target(path = '')
          base = path.match(/^\//) ? root_url : url
          uri = "#{base}/#{path}" 
          uri = URI.escape(URI.escape(uri), '[]')
          shell_quote(uri.gsub(/[?<>\*]/, ''))
        end

This code is escaping too much. If URL is properly entered (and it should be) then no need to escape it again.

This is a regression from 0.9 series that worked ok with URLs.


Related issues

related to Defect #2532: Cannot browse Subversion paths that contain spaces New 2009-01-19
related to Defect #5461: Cannot browse SVN Repositories with whitespaces in them Closed 2010-05-07

History

#1 Updated by Toshi MARUYAMA about 1 year ago

  • Subject changed from Repository URL not handled correctly to Subversion: Repository URL not handled correctly

#2 Updated by Toshi MARUYAMA about 1 year ago

  • Subject changed from Subversion: Repository URL not handled correctly to Subversion: Repository URL with space not handled correctly

Also available in: Atom PDF