[squid-dev] [PATCH] Ignore impossible SSL bumping actions, as intended and documented / bug 4237 fix

[Alex Rousskov]

On 08/11/2015 10:40 AM, Amos Jeffries wrote:
> On 11/08/2015 11:24 p.m., Tsantilas Christos wrote:

>> The reconnect is not yet implemented.


> Gah. So for the last year-ish it has been published in wiki
> SslPeekAndSplice page as an available config option. But is actually
> nothing more than an unused enum value ?
> 
> Great. Please fix *that*.

"reconnect" is not an [unused] BumpMode enum value (I only checked trunk
though). Sorry about my "terminate" typo in the previous response.


I am just trying to explain that the situation with "reconnect" is a
minor problem: Trunk code is fine. Squid.conf.documentation is fine.
AFAIK, no user has suffered from misinterpreting the wiki page. Yes, the
wiki page wording is unfortunate and should be changed. That's all!


If you reread your original reaction to the "reconnect is not yet
implemented" news, you will see that it is easy to misinterpret(?) your
reaction as a request to implement "reconnect" in order to fix some big
problem that we should be all very upset about. That
misinterpretation(?) is the reason I posted an explanation. There is
absolutely no need to turn this misunderstanding(?) into yet another
time-wasting thread.

For the future, avoiding unqualified pronouns in isolated
paragraph-sentences may help a lot. It is often very difficult to know
what "this", "that", or "it" refers to because a lot of things above an
isolated sentence can be "this" or "that". For example, saying "Please
remove reconnect documentation from the wiki page" would have saved a
lot of time. This exchange is just one of many such instances.


> 0. Complete removal of the action entry from wiki.

Done. I have removed wiki documentation for the unsupported "err" action
as well.

Alex.