Amos Jeffries

Results 156 comments of Amos Jeffries

Closing as superseded by PR #1238

Rebased to resolve conflicts and kick fresh build tests. Removed M-waiting-for-distro as Squid-6 requirement for C++17 support mandates support for the C++11 functionality this requires.

Status update: The branch has been updated, based on latest v7. It builds. TODO: * testing that it works as intended.

Tags exist in the release repository. https://github.com/squidadm/squid/tags

> > Tags exist in the release repository. https://github.com/squidadm/squid/tags > > @yadij, do you want me to copy those tags from that unofficial "release repository" into the official Squid Project...

@kinkie, you requested my review. But with the latest iteration of this I am waiting on confirmation that the mingw builds no longer need this compat code. IIRC squidclient.exe used...

IMO we have enough interest to keep it around, but not enough to waste our resources spending more time on this discussion. The shuffling (non-)change in this PR at least...

Currently requires PR #1560

FTR; 1) Squid current behaviour of doing a TLS "success" and presenting 5xx HTTP error page - is to avoid bugs in Chrome and Firefox handling of non-`HTTP/1.1 200 OK`...