This is an automated email from the ASF dual-hosted git repository. kgiusti pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/qpid-dispatch.git
commit a14b289b9df2e99329e8023c31afbc4ee7edf013 Author: Kenneth Giusti <[email protected]> AuthorDate: Mon Jan 4 13:29:42 2021 -0500 DISPATCH-1744: remove 100 Continue test on HTTP/1.0 server The python HTTPServer will occasionally incorrectly close the TCP connection after sending a 100-Continue response. This causes the test to fail. This failure is not due to the adaptor. This closes #942 --- src/adaptors/http1/http1_server.c | 6 ------ tests/system_tests_http1_adaptor.py | 8 +++----- 2 files changed, 3 insertions(+), 11 deletions(-) diff --git a/src/adaptors/http1/http1_server.c b/src/adaptors/http1/http1_server.c index 1e062c6..f020f70 100644 --- a/src/adaptors/http1/http1_server.c +++ b/src/adaptors/http1/http1_server.c @@ -761,9 +761,6 @@ static void _server_tx_buffers_cb(h1_codec_request_state_t *hrs, qd_buffer_list_ "[C%"PRIu64"][L%"PRIu64"] Sending %u octets to server", hconn->conn_id, hconn->out_link_id, len); qdr_http1_enqueue_buffer_list(&hreq->out_data, blist); - if (hreq == (_server_request_t*) DEQ_HEAD(hconn->requests)) { - _write_pending_request(hreq); - } } } @@ -782,9 +779,6 @@ static void _server_tx_stream_data_cb(h1_codec_request_state_t *hrs, qd_message_ "[C%"PRIu64"][L%"PRIu64"] Sending body data to server", hconn->conn_id, hconn->out_link_id); qdr_http1_enqueue_stream_data(&hreq->out_data, stream_data); - if (hreq == (_server_request_t*) DEQ_HEAD(hconn->requests)) { - _write_pending_request(hreq); - } } } diff --git a/tests/system_tests_http1_adaptor.py b/tests/system_tests_http1_adaptor.py index 237adca..57979cc 100644 --- a/tests/system_tests_http1_adaptor.py +++ b/tests/system_tests_http1_adaptor.py @@ -708,11 +708,9 @@ class Http1AdaptorOneRouterTest(TestCase): (RequestMsg("GET", "/GET/info_content_len", headers={"Content-Length": 0}), - [ResponseMsg(100, reason="Continue", - headers={"Blab": 1, "Blob": "?"}), - ResponseMsg(200, reason="OK", - headers={"Content-Type": "text/plain;charset=utf-8"}, - body=b'?')], + ResponseMsg(200, reason="OK", + headers={"Content-Type": "text/plain;charset=utf-8"}, + body=b'?'), ResponseValidator(expect_headers={'Content-Type': "text/plain;charset=utf-8"}, expect_body=b'?')), --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
