tree 8d9c09df174903be147d8c22e2bd538f59720c0e
parent 6486ff8cabb1495f8fc76d66fc37c66613773cf4
author Willy Tarreau <w@1wt.eu> 1660922443 +0200
committer Willy Tarreau <w@1wt.eu> 1662112081 +0200
encoding latin1

BUG/MINOR: httpclient: keep-alive was accidentely disabled

The servers were not set with default settings, meaning that a few
settings including the pool_max_delay were not set, thus disabling
connection pools, which is the cause of the fact that keep-alive was
disabled as reported in issue #1831. There might possibly be other
issues pending since all these fields were left to zero.

Note that this patch alone will not fix keep-alive because the applet
does not enforce SE_FL_NOT_FIRST and relies on the default http-reuse
safe, thus if servers are not shared, all requests are considered
first ones and do not reuse existing connections.

In 2.7, commit ecb40b2c3 ("MINOR: backend: always satisfy the first
req reuse rule with l7 retries") addressed this in a more elegant way
by fixing http-reuse to take into account the fact that properly
configured l7 retries provide exactly the capability that reuse safe
was trying to cover, and this patch is suitable for backporting.

This patch should be backported to 2.6 only.