Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-18 Thread Joe Orton
On Tue, Aug 17, 2010 at 06:00:58PM +0200, Plüm, Rüdiger, VF-Group wrote: I think you should use ((apr_table_get(r-headers_in, Content-Length) || apr_table_get(r-headers_in, Transfer-Encoding))) as we only want do_100_continue to be true *if* we have a request body, which means that

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-18 Thread Jim Jagielski
On Aug 18, 2010, at 8:26 AM, Joe Orton wrote: On Tue, Aug 17, 2010 at 06:00:58PM +0200, Plüm, Rüdiger, VF-Group wrote: I think you should use ((apr_table_get(r-headers_in, Content-Length) || apr_table_get(r-headers_in, Transfer-Encoding))) as we only want do_100_continue to be true

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Rainer Jung
On 17.08.2010 16:43, j...@apache.org wrote: Author: jim Date: Tue Aug 17 14:43:45 2010 New Revision: 986333 URL: http://svn.apache.org/viewvc?rev=986333view=rev Log: Further checks for non-body requests... Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_http.c Modified:

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Jim Jagielski
On Aug 17, 2010, at 10:55 AM, Rainer Jung wrote: On 17.08.2010 16:43, j...@apache.org wrote: Author: jim Date: Tue Aug 17 14:43:45 2010 New Revision: 986333 URL: http://svn.apache.org/viewvc?rev=986333view=rev Log: Further checks for non-body requests... Modified:

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Rainer Jung
On 17.08.2010 17:08, Jim Jagielski wrote: On Aug 17, 2010, at 10:55 AM, Rainer Jung wrote: On 17.08.2010 16:43, j...@apache.org wrote: Author: jim Date: Tue Aug 17 14:43:45 2010 New Revision: 986333 URL: http://svn.apache.org/viewvc?rev=986333view=rev Log: Further checks for non-body

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Jim Jagielski
On Aug 17, 2010, at 10:55 AM, Rainer Jung wrote: On 17.08.2010 16:43, j...@apache.org wrote: Author: jim Date: Tue Aug 17 14:43:45 2010 New Revision: 986333 URL: http://svn.apache.org/viewvc?rev=986333view=rev Log: Further checks for non-body requests... Modified:

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Rainer Jung
On 17.08.2010 17:22, Jim Jagielski wrote: On Aug 17, 2010, at 10:55 AM, Rainer Jung wrote: On 17.08.2010 16:43, j...@apache.org wrote: Author: jim Date: Tue Aug 17 14:43:45 2010 New Revision: 986333 URL: http://svn.apache.org/viewvc?rev=986333view=rev Log: Further checks for non-body

RE: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Plüm, Rüdiger, VF-Group
-Original Message- From: Jim Jagielski Sent: Dienstag, 17. August 2010 17:22 To: dev@httpd.apache.org Subject: Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c On Aug 17, 2010, at 10:55 AM, Rainer Jung wrote: On 17.08.2010 16:43, j

Re: svn commit: r986333 - /httpd/httpd/trunk/modules/proxy/mod_proxy_http.c

2010-08-17 Thread Jim Jagielski
as we only want do_100_continue to be true *if* we have a request body, My mistake for some reason I commented and coded the reverse, even though I knew what it should have been... Too much on my mind...