Subversion 1.7 libs now handle Redirect (http error 301), this breaks Tortoise 1.7.4.22459 repro browser

253 views
Skip to first unread message

Ken DeJager

unread,
Feb 10, 2012, 2:52:17 PM2/10/12
to us...@tortoisesvn.tigris.org
 
Subversion 1.7.2 now handles the http redirect for a repository moved 'permanently'.  It gives a message and redirects to the new URL, and longer reports error 301.
Svn's handling of redirect for a 'temporary' move is unchanged.
Tortoisesvn choses NOT to handle any redirects.
 
Unfortunately, svn 1.7's new behavior confuses Repository Browser, and a bogus folder appears.  Example:
 
I have a repository at http://localhost/svn/good_path
 
My httpd.conf has these intra-server redirects:
#
Redirect permanent /svn/bad_path_301      /svn/good_path
Redirect temp         /svn/bad_path_302      /svn/good_path

1. In Repository Browser, enter URL: http://localhost/svn/bad_path_302
Result is correct: "Unable to connect ... Repository moved temporarily to '//localhost/svn/good_path' (etc)".

2. In Repository Browser, enter URL: http://localhost/svn/bad_path_301
Expected behavior: "Unable to connect ... Repository moved permanently (etc)".
Incorrect result is:
 
http://localhost/svn/good_path
+  Branches
-  bad_path_301    (line is selected)
   +  Branches
   +  Tags
   + Trunk
+  Tags
+  Trunk
 
The 'bad_path_301' folder is incorrect and should not appear.
If I select any folder inside bad_path_301, such /bad_path_301/Branches, the other pane displays:  "'C:\ ... \Temp' is not a working copy".
The other folders seem OK.
Tortoise should either surrender to svn and correctly support its silent redirect, or else detect the redirect and report an error.  (I vote for the latter.)
 
Thanks,
Ken Dejager
 
 

Stefan Küng

unread,
Feb 11, 2012, 8:31:50 AM2/11/12
to us...@tortoisesvn.tigris.org
On 10.02.2012 20:52, Ken DeJager wrote:
>
> Subversion 1.7.2 now handles the http redirect for a repository moved
> 'permanently'. It gives a message and redirects to the new URL, and
> longer reports error 301.
> Svn's handling of redirect for a 'temporary' move is unchanged.
> Tortoisesvn choses NOT to handle any redirects.
>
> Unfortunately, svn 1.7's new behavior confuses Repository Browser, and a
> bogus folder appears. Example:
>
> I have a repository at http://localhost/svn/good_path
>
> My httpd.conf has these intra-server redirects:
> #
> Redirect permanent /svn/bad_path_301 /svn/good_path
> Redirect temp /svn/bad_path_302 /svn/good_path
>
> 1. In Repository Browser, enter URL: http://localhost/svn/bad_path_302
> Result is correct: "Unable to connect ... Repository moved temporarily
> to '//localhost/svn/good_path' (etc)".
>
> 2. In Repository Browser, enter URL: http://localhost/svn/bad_path_301
> Expected behavior: "Unable to connect ... Repository moved permanently
> (etc)".
> Incorrect result is:
>
> <http://localhost/svn/>http://localhost/svn/
> <http://localhost/svn/good_path>good_path

> + Branches
> - bad_path_301 (line is selected)
> + Branches
> + Tags
> + Trunk
> + Tags
> + Trunk
>
> The 'bad_path_301' folder is incorrect and should not appear.
> If I select any folder inside bad_path_301, such /bad_path_301/Branches,
> the other pane displays: "'C:\ ... \Temp' is not a working copy".
> The other folders seem OK.
> Tortoise should either surrender to svn and correctly support its silent
> redirect, or else detect the redirect and report an error. (I vote for
> the latter.)


I've opened issue #209 for this:
http://code.google.com/p/tortoisesvn/issues/detail?id=209

Stefan

--
___
oo // \\ "De Chelonian Mobile"
(_,\/ \_/ \ TortoiseSVN
\ \_/_\_/> The coolest Interface to (Sub)Version Control
/_/ \_\ http://tortoisesvn.net

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=2921173

To unsubscribe from this discussion, e-mail: [users-un...@tortoisesvn.tigris.org].

Reply all
Reply to author
Forward
0 new messages