]> git.ipfire.org Git - thirdparty/squid.git/commit
Docs: REQUIRED in ident_regex, proxy_auth_regex, ext_user_regex (#1818)
authorAlex Rousskov <rousskov@measurement-factory.com>
Mon, 20 May 2024 14:50:19 +0000 (14:50 +0000)
committerAmos Jeffries <yadij@users.noreply.github.com>
Thu, 23 May 2024 15:45:26 +0000 (03:45 +1200)
commit1e6d0b665e4ec37e30566a879a0e69f3d12b153e
treeabb98bad55e89e8d4a54a36a4e4b5e209b7356e7
parentfb0d1814dd1455f7f202f6018e895aa821763b01
Docs: REQUIRED in ident_regex, proxy_auth_regex, ext_user_regex (#1818)

The three ACLs were documented as matching any username when configured
with a parameter spelled "REQUIRED". Neither actually treated that
parameter specially -- all interpreted it as an ordinary regex.

This dangerous documentation bug was introduced in 2000 commit 145cf92
that added ident_regex and proxy_auth_regex support. It was then
duplicated in 2003 commit abb929f that added ext_user_regex support.

This minimal documentation fix does not imply that these ACLs should not
treat REQUIRED values specially. Enabling such special treatment
requires significant code changes, especially if we want to do that well
and without duplicating the corresponding code.
src/cf.data.pre