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

Re: svn commit: r34322 - in branches/http-protocol-v2/subversion: include mod_dav_svn

From: Greg Stein <gstein_at_gmail.com>
Date: Mon, 24 Nov 2008 17:30:14 -0800

On Mon, Nov 24, 2008 at 09:37, Ben Collins-Sussman <sussman_at_red-bean.com> wrote:
> On Sat, Nov 22, 2008 at 10:03 AM, Greg Stein <gstein_at_gmail.com> wrote:
>
>> A bit of a hack here. The request_rec is in the private data of the
>> resource. You can use that to access r->headers_out and set arbitrary
>> headers. Do that in the get_option() callback. We happen to know that
>> a body will always be sent in the OPTIONS request, so that function
>> will always be called. (and if an arbitrary client sends OPTIONS
>> without a body, then they won't get the extra headers)
>
> Um, well, guess what? It seems that ra_serf isn't sending a body in
> its OPTIONS request when I do an 'svn ls http://localhost/repos'. I
> just verified this by looking at the HTTP stream.
>
> So yeah, I made all the changes you suggested in get_option(), but
> get_option() isn't being called. :-/

Damn. I looked at ra_neon; didn't think ra_serf would be different.

Heh.

Answer: ra_serf should send a body :-)

(yes; the hack stench continues to spread... but this is easier than
telling people to get a new httpd (tho we may want to patch that
somehow))

Cheers,
-g

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-11-25 02:31:46 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.