BUG/MEDIUM: doc: Fix replace-path action description
authorChristopher Faulet <cfaulet@haproxy.com>
Wed, 2 Sep 2020 12:16:59 +0000 (14:16 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 2 Sep 2020 15:29:00 +0000 (17:29 +0200)
commit82c8332be058d79ce0b97151cd312ea5e43a4356
treed00e7d9c73fe3b9613158b5c574877b58b96febe
parent1fa0cc18e1c6dd5460492128b5b400d808f3c823
BUG/MEDIUM: doc: Fix replace-path action description

The description of the replace-path action does not reflect what the code
do. When the request path is replaced, the query-string is preserved. But the
documentation stated the query-string is part of the replacement, if any is
present. Most of time, when the doc and the code differ, the code is fixed. But
here, the replace-path action is pretty confusing because the set-path action is
only applied on the path. The query-string is left intact. And the path sample
fetch also ignores the query-string. In addition, the replace-path action is
quite recent. It was added in the 2.2. Thus, exceptionally, the documentation is
fixed instead.

Note that set-pathq and replace-pathq actions and pathq sample fetch will be
added to manipulate the path with the query-string.

This patch must be backported as far as 2.0.
doc/configuration.txt