Code review comment for ~sergiodj/ubuntu/+source/squid:squid-merge-4.13-1

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

Dropping the now accepted patch is fine.
Everything else stays the same as it was 15 days ago, which means it still is as ok as it was last time.

Changelog entries look good and I see no good chance to upstream more of our delta except that which you have already started.

http://squid-web-proxy-cache.1019090.n4.nabble.com/squid-announce-Squid-4-13-is-available-td4689798.html erads ok as well, the CVes are marked there as needed.

I'm slightly unsure on the TLS/1.3 change, but following upstream should be the right choice here.

+1 and let me know if you want me to sponsor it.

review: Approve

« Back to merge proposal