]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: http_ana: use scf to report term_evts in http_wait_for_request()
authorWilly Tarreau <w@1wt.eu>
Sun, 26 Apr 2026 20:01:59 +0000 (22:01 +0200)
committerWilly Tarreau <w@1wt.eu>
Mon, 27 Apr 2026 16:58:51 +0000 (18:58 +0200)
http_wait_for_request() improperly reports term events on the scb instead
of scf, causing some request parsing failures to possibly be reported as
response errors. This was introduced in 3.2 with commit 2dc02f75b1 ("MEDIUM:
tevt/stconn/stream: Add dedicated termination events for stream location")
so it must be backported there.

src/http_ana.c

index 9087dfa9e2792dcb0e6320285449d014a5858746..533613fd449e2c948c49fb43c93e11ceae680447 100644 (file)
@@ -348,7 +348,7 @@ int http_wait_for_request(struct stream *s, struct channel *req, int an_bit)
                _HA_ATOMIC_INC(&sess->fe_tgcounters->internal_errors);
        if (sess->li_tgcounters)
                _HA_ATOMIC_INC(&sess->li_tgcounters->internal_errors);
-       stream_report_term_evt(s->scb, strm_tevt_type_internal_err);
+       stream_report_term_evt(s->scf, strm_tevt_type_internal_err);
        goto return_prx_cond;
 
  return_bad_req:
@@ -357,7 +357,7 @@ int http_wait_for_request(struct stream *s, struct channel *req, int an_bit)
                _HA_ATOMIC_INC(&sess->fe_tgcounters->failed_req);
        if (sess->li_tgcounters)
                _HA_ATOMIC_INC(&sess->li_tgcounters->failed_req);
-       stream_report_term_evt(s->scb, strm_tevt_type_proto_err);
+       stream_report_term_evt(s->scf, strm_tevt_type_proto_err);
        /* fall through */
 
  return_prx_cond: