]> git.ipfire.org Git - thirdparty/squid.git/commit - src/acl/FilledChecklist.cc
projects / thirdparty / squid.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 279e60e) | patch
Fix %>ru for CONNECT requests (#299) M-staged-PR299
authorChristos Tsantilas <christos@chtsanti.net>
Thu, 4 Oct 2018 10:15:15 +0000 (10:15 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Sat, 6 Oct 2018 10:15:51 +0000 (10:15 +0000)
commit57a5679bae20e90ef73473e03327e37aa0263570
tree724c5fca17e68dc54aef8f07f2ac13e2ca69d911tree | snapshot
parent279e60ef962bd339920252f8084fbe88a0aa8a68commit | diff
Fix %>ru for CONNECT requests (#299)

Commit bec110e (a.k.a. v4 commit fbbd5cd5) broke CONNECT URI logging
because it incorrectly assumed that URI::absolute() supports all URIs.
As the result, Squid logged CONNECT URLs as "://host:port".

Also fixed a similar wrong assumption in ACLFilledChecklist::verifyAle()
which may affect URL-related ACL checks for CONNECT requests, albeit
only in already buggy cases where Squid warns about "ALE missing URL".

This is a Measurement Factory project
src/AccessLogEntry.cc diff | blob | blame | history
src/acl/FilledChecklist.cc diff | blob | blame | history
Mirror of https://github.com/squid-cache/squid.git