From a625b03e834704c5683e6a3abd09f9163ac8adc6 Mon Sep 17 00:00:00 2001 From: William Lallemand Date: Thu, 17 Mar 2022 14:45:46 +0100 Subject: [PATCH] BUG/MINOR: httpclient: only check co_data() instead of HTTP_MSG_DATA Checking msg >= HTTP_MSG_DATA was useful to check if we received all the data. However it does not work correctly in case of errors because we don't reach this state, preventing to catch the error in the httpclient. The consequence of this problem is that we don't get the status code of the error response upon an error. Fix the issue by only checking co_data(). Must be backported to 2.5. --- src/http_client.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/http_client.c b/src/http_client.c index f5a29934d..c449418ed 100644 --- a/src/http_client.c +++ b/src/http_client.c @@ -645,7 +645,6 @@ static void httpclient_applet_io_handler(struct appctx *appctx) struct stream *s = si_strm(si); struct channel *req = &s->req; struct channel *res = &s->res; - struct http_msg *msg = &s->txn->rsp; struct htx_blk *blk = NULL; struct htx *htx; struct htx_sl *sl = NULL; @@ -735,7 +734,7 @@ static void httpclient_applet_io_handler(struct appctx *appctx) case HTTPCLIENT_S_RES_STLINE: /* copy the start line in the hc structure,then remove the htx block */ - if (!co_data(res) || (msg->msg_state < HTTP_MSG_DATA)) + if (!co_data(res)) goto more; htx = htxbuf(&res->buf); if (!htx) @@ -774,7 +773,7 @@ static void httpclient_applet_io_handler(struct appctx *appctx) { struct http_hdr hdrs[global.tune.max_http_hdr]; - if (!co_data(res) || (msg->msg_state < HTTP_MSG_DATA)) + if (!co_data(res)) goto more; htx = htxbuf(&res->buf); if (!htx) @@ -839,7 +838,7 @@ static void httpclient_applet_io_handler(struct appctx *appctx) * The IO handler removes the htx blocks in the response buffer and * push them in the hc->res.buf buffer in a raw format. */ - if (!co_data(res) || (msg->msg_state < HTTP_MSG_DATA)) + if (!co_data(res)) goto more; htx = htxbuf(&res->buf);