upstream/haproxy
1
0
Fork 0
mirror of https://github.com/haproxy/haproxy.git synced 2026-06-19 14:31:44 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

BUG/MEDIUM: http-ana: Don't ignore L7 retry errors

Browse source 
with L7 retries are configured, when the max number of retries is reached
the error must be reported to the client. However, when it was an abort on a
reused connections, the client connection is silently closed. While it is
expected without L7 retries, to let the client retries on its own, it is
unexepcted with L7 retries.

So let's fix it by ignoring the SF_SRV_REUSED flag on the stream when a L7
retry fails. This way, a 502/425 will be reported to the client.

This patch should help to fix the issue #3414. It must be backported to all
supported versions.
This commit is contained in:
Christopher Faulet 2026-06-12 15:57:50 +02:00
parent 7bfa568e27
commit 292b07270c
1 changed files with 12 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

20
src/http_ana.c
View file

@ -1265,7 +1265,7 @@ int http_wait_for_response(struct stream *s, struct channel *rep, int an_bit)
struct htx *htx;
struct connection *srv_conn;
struct htx_sl *sl;
int n;
int n, l7_retry_failed = 0;
DBG_TRACE_ENTER(STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA, s, txn, msg);
@ -1306,19 +1306,22 @@ int http_wait_for_response(struct stream *s, struct channel *rep, int an_bit)
(!conn || conn->err_code != CO_ER_SSL_EARLY_FAILED)) {
if (co_data(rep) || do_l7_retry(s, s->scb) == 0)
return 0;
l7_retry_failed = 1;
}
/* Perform a L7 retry on empty response or because server refuses the early data. */
if ((txn->flags & TX_L7_RETRY) &&
(s->be->retry_type & PR_RE_EARLY_ERROR) &&
conn && conn->err_code == CO_ER_SSL_EARLY_FAILED &&
do_l7_retry(s, s->scb) == 0) {
DBG_TRACE_DEVEL("leaving on L7 retry",
STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA, s, txn);
return 0;
conn && conn->err_code == CO_ER_SSL_EARLY_FAILED) {
if (do_l7_retry(s, s->scb) == 0) {
DBG_TRACE_DEVEL("leaving on L7 retry",
STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA, s, txn);
return 0;
}
l7_retry_failed = 1;
}
if (s->flags & SF_SRV_REUSED)
if (!l7_retry_failed && (s->flags & SF_SRV_REUSED))
goto abort_keep_alive;
if (s->be_tgcounters)
@ -1416,9 +1419,10 @@ int http_wait_for_response(struct stream *s, struct channel *rep, int an_bit)
STRM_EV_STRM_ANA|STRM_EV_HTTP_ANA, s, txn);
return 0;
}
l7_retry_failed = 1;
}
if (s->flags & SF_SRV_REUSED)
if (!l7_retry_failed && (s->flags & SF_SRV_REUSED))
goto abort_keep_alive;
if (s->be_tgcounters)