From 1741bc4bf0f8bb5014b37e44b98d23664b0ec5b9 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Tue, 22 Jul 2025 15:17:50 +0200 Subject: [PATCH] BUG/MINOR: httpclient-cli: Don't try to dump raw headers in HTX mode In the CLI I/O handler interacting with the HTTP client, we must not try to push raw headers in HTX mode, because there is no raw data in this mode. This prevent the HTX dump at the end of the I/O handle. It is a 3.3-specific issue. No backport needed. --- src/httpclient_cli.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/httpclient_cli.c b/src/httpclient_cli.c index 1b41c9214..1f96c8369 100644 --- a/src/httpclient_cli.c +++ b/src/httpclient_cli.c @@ -197,8 +197,8 @@ static int hc_cli_io_handler(struct appctx *appctx) } if (ctx->flags & HC_F_RES_HDR) { - chunk_reset(&trash); if (!ctx->is_htx) { + chunk_reset(&trash); hdrs = hc->res.hdrs; for (hdr = hdrs; isttest(hdr->v); hdr++) { if (!h1_format_htx_hdr(hdr->n, hdr->v, &trash)) @@ -206,9 +206,9 @@ static int hc_cli_io_handler(struct appctx *appctx) } if (!chunk_memcat(&trash, "\r\n", 2)) goto too_many_hdrs; + if (applet_putchk(appctx, &trash) == -1) + goto more; } - if (applet_putchk(appctx, &trash) == -1) - goto more; ctx->flags &= ~HC_F_RES_HDR; } -- 2.47.2