]> git.ipfire.org Git - thirdparty/squid.git/commit
Use ERR_ACCESS_DENIED for HTTP 403 (Forbidden) errors (#1899)
authorAlex Rousskov <rousskov@measurement-factory.com>
Fri, 20 Sep 2024 09:35:04 +0000 (09:35 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Fri, 20 Sep 2024 18:47:12 +0000 (18:47 +0000)
commitfdc5bf76a3ce83fee72e6b871f0df30504b342df
tree024cc9000895ea3ad4f22143e3007bd023663b61
parent84f5cdd658e3e8362ef5a38958ac3a4b3055e763
Use ERR_ACCESS_DENIED for HTTP 403 (Forbidden) errors (#1899)

... when request authentication fails. Do not use
ERR_CACHE_ACCESS_DENIED for those "permanent" errors.

Default ERR_CACHE_ACCESS_DENIED is meant for cases where the user is
likely to eventually gain access (e.g., by supplying credentials). Its
default text says "not currently allowed... until you have authenticated
yourself". When the error page was added in 1998 commit cb69b4c7 it was
only used for HTTP 407 errors. The same logic was preserved when that
code was refactored in 1999 commit 1cfdbcf0, but exceptions started to
creep in, perhaps accidentally, since 2011 when HTTP 403 case was added
in commit 2f1431ea that introduced USE_AUTH macro. 2011 commit 21512911
added a similar "not possible to authenticate" SslBump case.

Other HTTP 403 (Forbidden) cases already use ERR_ACCESS_DENIED or a
similar "permanent" error (e.g., ERR_FORWARDING_DENIED or ERR_TOO_BIG).

It is still possible to customize the returned error page via deny_info.
src/client_side_request.cc