On Thu, 15 Nov 2007, C. Michael Pilato wrote:
> Stefan Küng wrote:
>> C. Michael Pilato wrote:
>>> David Glasser wrote:
>>>> On Nov 15, 2007 12:08 PM, C. Michael Pilato <cmpilato@collab.net> wrote:
>>>>> Stefan Küng wrote:
>>>>>> Hi,
>>>>>>
>>>>>> The attached patch removes the fixed user-agent strings used in the
>>>>>> svn
>>>>>> library and replaces them with a define. That way, svn clients can set
>>>>>> that define to a custom user-agent string when compiling the library.
>>>>>>
>>>>>> If the client does not define a custom user-agent string, a default
>>>>>> ("SVN/") is used (the same as before).
>>>>> Stefan, are you aware of the fact that mod_dav_svn actually pays
>>>>> attention
>>>>> the User-Agent header, specifically searching for those which begin
>>>>> with
>>>>> "SVN/"? This is used in the LOCK handling code and, I think, the GET
>>>>> response handler, too.
>>>> How about adding a parenthesized identifier, like web browsers do?
>>>
>>> +1.
>>>
>>> We just need a #define for CUSTOM_USER_AGENT that our libraries use
>>> like so:
>>>
>>> "User-Agent: SVN/" SVN_VERSION " (" CUSTOM_USER_AGENT ")"
>>>
>>> (Or maybe we just ask that the CUSTOM_USER_AGENT value include the
>>> space and
>>> parens ... whatever's easiest.)
>>
>> See attached patch. I haven't checked, but if mod_dav_svn only checks
>> the beginning of the user-agent string, then this should work.
>
> from mod_dav_svn/repos.c:
>
> if (val && (ap_strstr_c(val, "SVN/") == val))
>
Why not add a function which sets (or just modifies) the user agent?
Clients which use the shared libraries would then also be able to change
it.
Just my 2 cents.
Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Nov 16 03:21:39 2007