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

Re: svn diff auth bug

From: Andreas J. Koenig <andreas.koenig_at_anima.de>
Date: 2002-12-10 13:49:11 CET

It's nearly two months since...

>>>>> On 18 Oct 2002 08:49:55 -0500, Ben Collins-Sussman <sussman@collab.net> said:

> Rafael Garcia-Suarez <raphel.garcia-suarez@hexaflux.com> writes:
>> I've an http svn server that requires a valid user for all http methods,
>> and a working copy with cached auth tokens. But the command
>> "svn diff -r <rev#> targets" still asks me for a password. This behavior
>> disappears if I make the http server accessible for anyone for the GET,
>> PROPFIND, OPTIONS and REPORT methods. (Apparently there is no issue
>> filled for this bug yet.)
>> I could try to fix this by myself if I find some time -- has someone
>> any tuits on where to look ?

> We have three different code paths for 'svn diff':

> * working compared to text-base (no network)

> * working compared to URL

> * URL compared to URL

> The latter two cases are fancy diff editors -- one lives in libsvn_wc,
> the other in libsvn_client, I believe. They both involve calling
 RA-> get_file() into a temporary file, which is then diffed against
> either your working file, or another temp file.

> So the issue here, I'm guessing, is that for the 2nd case (which
> you're using) RA->get_file() isn't using any auth in the working copy.
> For our 3rd codepath, this is expected behavior -- it assumes that no
> working copy exists. But for the 2nd codepath, we should probably be
> opening the RA session differently, i.e. with access to a working copy
> (and thus its cache.)

I just discovered that this issue still exists but is not filed as an
issue. Shouldn't it either be fixed or filed?

To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Dec 10 13:50:07 2002

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