Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-25 Thread Eric Jacquot
Sorry for my fuckin' iphone... I really apologise ... -- Eric Jacquot Le 26 sept. 2010 ` 00:00, "Eric Jacquot" a icrit : > Le 23 sept. 2010 ` 08:57, "Sebastian Reitenbach" > a icrit : > > On Wednesday 22 September 2010 > 11:19:28 pm Landry Breuil wrote: > On Wed, Sep 22, 2010 at 05:20:04PM +0

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-25 Thread Eric Jacquot
Le 23 sept. 2010 ` 08:57, "Sebastian Reitenbach" a icrit : On Wednesday 22 September 2010 11:19:28 pm Landry Breuil wrote: On Wed, Sep 22, 2010 at 05:20:04PM +0200, Sebastian Reitenbach wrote: Alexander Hall wrote: I don't have any actual interest in the change myself, nor the time to test

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-23 Thread Sebastian Reitenbach
On Wednesday 22 September 2010 11:19:28 pm Landry Breuil wrote: > On Wed, Sep 22, 2010 at 05:20:04PM +0200, Sebastian Reitenbach wrote: > > Alexander Hall wrote: > > > I don't have any actual interest in the change myself, nor the time to > > > test it, but now at least the diff has the fixes I exp

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-22 Thread Landry Breuil
On Wed, Sep 22, 2010 at 05:20:04PM +0200, Sebastian Reitenbach wrote: > Alexander Hall wrote: > > I don't have any actual interest in the change myself, nor the time to > > test it, but now at least the diff has the fixes I expected to see > > regarding my prior concerns. > > > > Anyone else wh

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-22 Thread Sebastian Reitenbach
Alexander Hall wrote: > I don't have any actual interest in the change myself, nor the time to > test it, but now at least the diff has the fixes I expected to see > regarding my prior concerns. > Anyone else who would test or comment on this one? Tested so far by me: telnetting to apache, and

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-13 Thread Alexander Hall
I don't have any actual interest in the change myself, nor the time to test it, but now at least the diff has the fixes I expected to see regarding my prior concerns. /Alexander On 09/13/10 19:50, Sebastian Reitenbach wrote: > Again, > > as I was pointed out the patch was borked, and actually do

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-13 Thread Sebastian Reitenbach
Again, as I was pointed out the patch was borked, and actually documentation missing, here is a new one, including the documentation. Sebastian On Monday, September 13, 2010 18:26 CEST, Sebastian Reitenbach wrote: > Hi, > > attached a new revision of the patch. Please comment, and let me

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-13 Thread Sebastian Reitenbach
Hi, attached a new revision of the patch. Please comment, and let me know whether its OK to add or still sth. wrong. thanks, Sebastian Alexander Hall wrote: > On 09/12/10 18:20, Sebastian Reitenbach wrote: > >> -table *tbl = (hdr->do_err ? r->err_headers_out : r->headers_out); >> +tab

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-13 Thread Sebastian Reitenbach
Hi, thanks for taking a look. On Sunday 12 September 2010 09:33:44 pm you wrote: > On 09/12/10 18:20, Sebastian Reitenbach wrote: > > -table *tbl = (hdr->do_err ? r->err_headers_out : r->headers_out); > > +table *tbl; > > +switch (hdr->inout) { > > +case hdrs_out: > > + tbl =

Re: patch to add RequestHeader directive to httpd mod_headers.c

2010-09-12 Thread Alexander Hall
On 09/12/10 18:20, Sebastian Reitenbach wrote: > -table *tbl = (hdr->do_err ? r->err_headers_out : r->headers_out); > +table *tbl; > +switch (hdr->inout) { > +case hdrs_out: > + tbl = r->headers_out; > + break; > +case hdrs_in: > + tbl = r->headers_in; > + b

patch to add RequestHeader directive to httpd mod_headers.c

2010-09-12 Thread Sebastian Reitenbach
Hi, my first time I submit a patch which not goes to the ports. I have a web based groupware port in the queue, that would benefit from the patch below. The patch is against mod_headers.c and adds handling of RequestHeader directive in the apache configuration, in order to manipulate the reque