[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] Fix links generated by ?r=REV

From: Martin Furter <mf_at_rola.ch>
Date: Tue, 19 May 2009 17:26:16 +0200 (CEST)

On Thu, 14 May 2009, C. Michael Pilato wrote:

> Ben Collins-Sussman wrote:
>> On Thu, May 14, 2009 at 3:18 PM, Branko Cibej <brane_at_xbc.nu> wrote:
>>
>>> IMHO he should use the exact same query string as was on the original
>>> URL. Anything else is guesswork, no?
>>
>> Er, this is tricky. Suppose I type in "http://host/some/dir?p=x&r=y".
>>
>> This wil locate dir_at_p, then follow the directory back in time to
>> revision y (where it might exist at a different location).
>>
>> The children displayed in the output are therefore all being displayed
>> at revision Y, and at some (possibly new) path. My first instinct is
>> to point out that they should all therefore have the ?p=y bit attached
>> to their hrefs, but this doesn't take into account the possibly new
>> parental path. :-/
>
> You're right, this is tricky. But I think the solution is as simple as
> behaving *literally* in the way we interpret the inputs semantically:
> respond to PATH?r=REV[&p=PEGREV] with a "301 Redirect" whose destination is
> the resolved, real location (path + rev) -- that is, REALPATH?p=PEGREV.

Shouldn't this be a "302 Moved Temporarily" since if r=HEAD it can point
to a different path later if it is replaced by something else?

(I guess if REV is not HEAD a 301 is perfectly fine.)

Martin

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2313950
Received on 2009-05-19 17:26:41 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.