]> 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)
committerSquid Anubis <squid-anubis@squid-cache.org>
Wed, 22 May 2024 06:42:36 +0000 (06:42 +0000)
commita4d88710bc5ccd5547ccae2c0bed2db7829402cc
treea6c43ee8281d2823223712f9951183d3e7edaa6e
parentdbbd46d97b19f44d34e16c92a070be51444c58a3
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