BUG/MINOR: http-act: set-status() must check the response message, not the request

action_http_set_status() checks for soft rewrite on the request message
by mistake instead of the response message. This could possibly cause a
rewrite failure when soft rewrite is enabled since it will not be seen
there, though the impact is extremely low. It can be backported.
This commit is contained in:
Willy Tarreau 2026-05-11 15:01:30 +02:00
parent 8941cc5f6d
commit cbdbc96e36

View file

@ -656,7 +656,7 @@ static enum act_return action_http_set_status(struct act_rule *rule, struct prox
if (s->sv_tgcounters)
_HA_ATOMIC_INC(&s->sv_tgcounters->failed_rewrites);
if (!(s->txn.http->req.flags & HTTP_MSGF_SOFT_RW)) {
if (!(s->txn.http->rsp.flags & HTTP_MSGF_SOFT_RW)) {
if (!(s->flags & SF_ERR_MASK))
s->flags |= SF_ERR_PRXCOND;
return ACT_RET_ERR;