Mailing List Archive: Apache: Bugs

[Bug 47087] Incorrect request body handling with Expect: 100-continue if the client does not receive a transmitted 300 or 400 response prior to sending its body

Index | Next | Previous | View Threaded

bugzilla at apache

Nov 2, 2009, 7:36 AM

Post #1 of 5 (230 views)
Permalink

[Bug 47087] Incorrect request body handling with Expect: 100-continue if the client does not receive a transmitted 300 or 400 response prior to sending its body

https://issues.apache.org/bugzilla/show_bug.cgi?id=47087

--- Comment #4 from Nick Kew <nick[at]webthing.com> 2009-11-02 07:36:25 UTC ---
Created an attachment (id=24460)
--> (https://issues.apache.org/bugzilla/attachment.cgi?id=24460)
Experimental patch.

I'm attaching a patch that should fix this by closing the connection, with the
proviso that r->expecting_100 is set (a check for the request header itself at
this point would be slower). If you have a testbed for provoking this
behavior, it would be great if you could test the patch against it.

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe[at]httpd.apache.org
For additional commands, e-mail: bugs-help[at]httpd.apache.org

bugzilla at apache

Nov 2, 2009, 7:45 AM

Post #2 of 5 (215 views)
Permalink

[Bug 47087] Incorrect request body handling with Expect: 100-continue if the client does not receive a transmitted 300 or 400 response prior to sending its body [In reply to]

https://issues.apache.org/bugzilla/show_bug.cgi?id=47087

--- Comment #5 from Nick Kew <nick[at]webthing.com> 2009-11-02 07:45:01 UTC ---
Scrap that: it's not going to work because we don't set r->status to
HTTP_CONTINUE. Need to fix up the two points where an interim response is sent
so we have a test that works.

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe[at]httpd.apache.org
For additional commands, e-mail: bugs-help[at]httpd.apache.org

bugzilla at apache

Nov 2, 2009, 8:13 AM

Post #3 of 5 (215 views)
Permalink

[Bug 47087] Incorrect request body handling with Expect: 100-continue if the client does not receive a transmitted 300 or 400 response prior to sending its body [In reply to]

https://issues.apache.org/bugzilla/show_bug.cgi?id=47087

Nick Kew <nick[at]webthing.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
Attachment #24460|0 |1
is obsolete| |

--- Comment #6 from Nick Kew <nick[at]webthing.com> 2009-11-02 08:13:22 UTC ---
Created an attachment (id=24461)
--> (https://issues.apache.org/bugzilla/attachment.cgi?id=24461)
Amended experimental patch

OK, anyone in a position to test-drive the amended patch is encouraged to do
so. This one unsets r->expecting_100 as soon as an interim response is sent
(which might want further thought) but best to test whether it fixes this bug
first!

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe[at]httpd.apache.org
For additional commands, e-mail: bugs-help[at]httpd.apache.org

bugzilla at apache

Nov 3, 2009, 5:25 AM

Post #4 of 5 (200 views)
Permalink

[Bug 47087] Incorrect request body handling with Expect: 100-continue if the client does not receive a transmitted 300 or 400 response prior to sending its body [In reply to]

https://issues.apache.org/bugzilla/show_bug.cgi?id=47087

--- Comment #7 from Stuart A. Malone <samalone[at]llamagraphics.com> 2009-11-03 05:25:16 UTC ---
Nick, the patch is working for me. I applied it to the httpd-2.2.14 sources and
re-ran my .NET Framework test code that combines PROPFIND, digest
authentication, and 100-continue. Before the patch, the test would fail if the
PROPFIND contained a body. After the patch, the PROPFIND succeeds both with and
without a body. It's a very limited test, but so far, so good.

Thanks for working on this.

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe[at]httpd.apache.org
For additional commands, e-mail: bugs-help[at]httpd.apache.org

bugzilla at apache

Nov 3, 2009, 5:28 AM

Post #5 of 5 (200 views)
Permalink

[Bug 47087] Incorrect request body handling with Expect: 100-continue if the client does not receive a transmitted 300 or 400 response prior to sending its body [In reply to]

https://issues.apache.org/bugzilla/show_bug.cgi?id=47087

Stuart A. Malone <samalone[at]llamagraphics.com> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |samalone[at]llamagraphics.com

--
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: bugs-unsubscribe[at]httpd.apache.org
For additional commands, e-mail: bugs-help[at]httpd.apache.org

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact lists@gossamer-threads.com