On 08/23/2016 07:59 AM, Marcus Kool wrote: > On 08/23/2016 12:44 AM, Alex Rousskov wrote: >> On 08/22/2016 08:14 PM, Marcus Kool wrote: >>> When I think I am done, I will let you know for a review. >> It is best to commit all your intended changes at once (if at all) >> rather than to use the public page as a scratch pad -- folks read what >> you commit. > I am aware of this and always leave a page in a consistent state. FWIW, I would aim much higher than consistency. I am waiting for your signal to start reviewing the already published changes. BTW, please note that you blanket change of letters SSL to letters TLS makes wiki diffs nearly useless for review. It is probably too late to undo that, but please avoid/delay similar non-changes in the future if you want to facilitate reviews. > I managed to get 2 CONNECTs to the URL rewriter Glad I was not imagining things :-). > But the 2 CONNECTs have both an IP address. > The %ssl::>sni macro does not expand in url_rewrite_extras but > expands fine in the logformat of Squid 3.5.20. > Can we call that a bug? I would. Trunk r14293 (and v3.5 r13913) fixed a similar bug a year ago, but this may be a separate problem or a regression. Alex. _______________________________________________ squid-users mailing list squid-users@xxxxxxxxxxxxxxxxxxxxx http://lists.squid-cache.org/listinfo/squid-users
