]> 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 21:14:57 +0000 (09:14 +1200)
commit2e4fb2dfacb422c1611c20aeee3d924d0a491dc6
tree7de11ca43d59b4c5b8551ac92f08b7d1c46d45c2
parentd63fb353ee81791e03137d4aad7435890e3cdf7c
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