]> git.ipfire.org Git - thirdparty/squid.git/commit
Forget non-peer access details (#1378)
authorEduard Bagdasaryan <eduard.bagdasaryan@measurement-factory.com>
Fri, 16 Jun 2023 15:10:59 +0000 (15:10 +0000)
committerSquid Anubis <squid-anubis@squid-cache.org>
Sat, 17 Jun 2023 14:31:56 +0000 (14:31 +0000)
commite37329828e58a274b01ef4276d96607bf190f8a3
tree770f6786213d47ba0477391d6baf925da64e7a9f
parentb91a85582f3781f3105d09b9221668519b6dffb6
Forget non-peer access details (#1378)

We were using the CachePeer class to record the address of each non-peer
sending us certain unwanted ICP responses, along with the number of such
responses and a histogram of associated ICP opcodes. Since 1997 commit
e102ebd, these historical records were accumulated without a limit and
linearly searched, endangering a Squid instance that used ICP. Using
CachePeer to store this non-cache_peer information also complicated
cache_peer code.

This change removes these records and the corresponding mgr:non_peers
report, leaving just the cache.log warning about unexpected messages.
The warning is useful because these ICP messages from non-peers indicate
a cache hierarchy misconfiguration or a fairly sophisticated attack.

This is the simplest fix that minimizes Squid and developer resources
spent on handling these errors.
doc/release-notes/release-7.sgml.in
src/CachePeer.h
src/cf.data.pre
src/neighbors.cc