]> git.ipfire.org Git - thirdparty/squid.git/commit
Bump SSL client on [more] errors encountered before ssl_bump evaluation
authorChristos Tsantilas <chtsanti@users.sourceforge.net>
Mon, 6 Feb 2017 16:52:12 +0000 (18:52 +0200)
committerChristos Tsantilas <chtsanti@users.sourceforge.net>
Mon, 6 Feb 2017 16:52:12 +0000 (18:52 +0200)
commit93046e079ada1c93904f800683220b7fd544ea51
tree67202e8c2ca38922a1f79dd40ed2634e6dd1a651
parent8527bed1346ad975fda5df8c599f698b08dd13d6
Bump SSL client on [more] errors encountered before ssl_bump evaluation

... such as ERR_ACCESS_DENIED with HTTP/403 Forbidden triggered by an
http_access deny rule match.

The old code allowed ssl_bump step1 rules to be evaluated in the
presence of an error. An ssl_bump splicing decision would then trigger
the useless "send the error to the client now" processing logic instead
of going down the "to serve an error, bump the client first" path.

Furthermore, the ssl_bump evaluation result itself could be surprising
to the admin because ssl_bump (and most other) rules are not meant to be
evaluated for a transaction in an error state. This complicated triage.

Also polished an important comment to clarify that we want to bump on
error if (and only if) the SslBump feature is applicable to the failed
transaction (i.e., if the ssl_bump rules would have been evaluated if
there were no prior errors). The old comment could have been
misinterpreted that ssl_bump rules must be evaluated to allow an
"ssl_bump splice" match to hide the error.

This is a Measurement Factory project.
src/client_side_request.cc