On Fri, Mar 14, 2008 at 12:30 PM, C. Michael Pilato <cmpilato_at_collab.net> wrote:
> Karl Fogel wrote:
> > Alexander Kitaev <Alexander.Kitaev_at_svnkit.com> writes:
> >> There is the following code in subversion/mod_dav_svn/repos.c:1643
> >>
> >> if (val && (ap_strstr_c(val, "SVN/") == val))
> >> {
> >> repos->is_svn_client = TRUE;
> >>
> >> /* Client capabilities are self-reported. There is no
> >> guarantee the client actually has the capabilities it says
> >> it has, we just assume it is in the client's interests to
> >> report accurately. Also, we only remember the capabilities
> >> the server cares about (even though the client may send
> >> more than that). */
> >> ....
> >> }
> >>
> >> SVNKit honestly sends "SVNKit ..." User-Agent header and as a result
> >> capabilities reported by SVNKit are ignored by mod_dav_svn. Could the
> >> condition above be changed to:
> >>
> >> if (val && (ap_strstr_c(val, "SVN") == val))
> >>
> >> So far we have to use SVN/Kit for User-Agent HTTP header value.
> >> Thanks!
> >
> > Hmmm, thanks for pointing this out.
> >
> > Your suggestion seems reasonable to me. It's pretty likely that
> > anything starting with "SVN" is SVN-related, and wants to be treated as
> > an SVN client. But I'm not well-versed in the relevant standards here;
> > does anyone know more? (Let's give it a day or two, since the change is
> > trivial and easy to backport to the 1.5.x line before rc1 comes out.)
>
> -0. Why doesn't SVNKit use:
>
> SVN/x.y.z SVNKit/a.b.c
>
> which would roughly translate as "This is a Subversion client compatible
> with Subversion x.y.z and using SVNKit a.b.c as the implementation."
Something like that sounds good. I think we should make this format
be supported by the new svn_ra_get_client_string thing too.
--dave
--
David Glasser | glasser@davidglasser.net | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-03-14 20:34:32 CET