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

Re: [PATCH] Accept --force for switch --relocate was Re: Possible SVN Relocate bug

From: Peter Lundblad <plundblad_at_google.com>
Date: 2007-03-14 09:32:26 CET

Justin Erenkrantz writes:
> On 3/14/07, Peter Lundblad <plundblad@google.com> wrote:
> > > Allow switch --relocate to accept --force to allow unreadable roots/UUIDs to
> > > be switched.
> > >
> >
> > This sounds a bit dangerous to me, but that might be OK, since --force
> > implies that. But, isnce I didn't follow the thread, can you explain
> > what the problem is that this wants to solve?
>
> Imagine that mod_authz_svn is set up to deny access to / - but they
> have access to at least some other sub-directory. Hence, the UUID
> check fails with a 403. --force seems like a reasonable workaround
> here - unless you have other ideas? -- justin

What about changing the validator signature to take an expected repo
root instead of this root flag. (I guess I didn't think of this scenario
when implementing this, or I'm missin something now;). Or isn't it possible
to do svn_ra_get_repos_root on an unreadable root (I hope it is).

This would avoid the problem altogether without the user having to run with
--force.

Thanks,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Mar 14 09:33:12 2007

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