]> git.ipfire.org Git - thirdparty/squid.git/commit
Fix PeerConnector handling of last-resort callback requirements
authorAlex Rousskov <rousskov@measurement-factory.com>
Thu, 15 Jul 2021 18:34:43 +0000 (14:34 -0400)
committerAlex Rousskov <rousskov@measurement-factory.com>
Fri, 16 Jul 2021 17:33:09 +0000 (13:33 -0400)
commit67c1efe95a00b964acbccdce2dca4fef41497a8a
treed170fa1c4fdeddabed24f2bacd680b005c5399eb
parent03865984c1e19cec8b8efdabea9c4c57f5d0084d
Fix PeerConnector handling of last-resort callback requirements

... and simplify the corresponding code.

Events like handleStopRequest() and callException() stop the job but
should not be reported as a BUG (e.g., it would be up to the
callException() to decide how to report the caught exception). There
might be (or there will) be other, similar cases where the job is
stopped prematurely for some non-BUG reason beyond swanSong() knowledge.

The existence of non-bug cases does not mean there could be no bugs
worth reporting here, but until they can be identified more reliably
than all these benign/irrelevant cases, reporting no BUGs is a (much)
lesser evil.
src/security/PeerConnector.cc
src/security/PeerConnector.h