Bert Huijben wrote on Mon, Nov 14, 2011 at 15:25:33 +0100:
> >   client: ( check-path ( 29:/trunk/service/public ( 3155 ) ) )
> >   server: ( success ( ( ) 0: ) )
> >   server: ( success ( none ) )
> > 
> > Interestingly, if the check-path argument is changed to an existing path
> > below "/trunk/service/public" (such as "/subdir"), then it works.
> > 
> > In other words: it seems that svnserve 1.7 considers the argument to
> > check-path relative to the URL that was used to connect, whereas
> > svnserve 1.6 considered it absolute to the repository.
> > 
> > Can you confirm this? Is it a problem in Subversion or Svnkit?
> 
> The documentation of this function says that it should be relative to the
> session, as all path arguments to ra commands.
> 
> This probably worked ok, because we happened to join the path using
> svn_path_join() which ignores its first argument when the second argument
> starts with a '/'.
> 
> In Subversion 1.7 we improved the checking by ensuring that the argument is
> a valid relative path (a relpath), which would remove the initial '/', so
> then the argument is mapped as a child of the session.
> 
> I would call this a bug in SvnKit, as they should have passed valid relative
> paths...

+1

> But maybe somebody else thinks differently about this.
> 
>       Bert
> 

Reply via email to