pponakan
pponakan
> Thank you very much for this contribution! While the current implementation cannot be accepted (for the reasons identified in specific change requests), it sounds like a useful feature that...
> Apologies if I still have any formatting issues or am not following some specific coding guidelines. I was not able to get the scripts/format-cpp.pl script to test my code...
Thanks @rousskov! I will run a canary test with this diff in our env to verify it still works as the initial patch. A simple test would be to reduce...
Thanks @rousskov, the diff looks fine IMO. I will use this to build a new image for our canary test. Will keep you posted on how the test is doing...
> Thanks. BTW when I pull this branch and attempt to run autoconf, I get the following error on an ubuntu system. I am trying to build squid from the...
Thanks @yadij, I had forgotten to run ./bootstrap.sh, that fixed the build. I am on ubuntu 20.04. However while running tests with a small local_port range I couldnt find any...
Update on what I have been able to test. I have confirmed my build steps are pulling in the specific commits into the build image. I went back to the...
Can ignore the errors mentioned on my previous message. They donot appear if the local_port range is reduced after squid with 30 workers is brought up. Confirmed latest commit does...
FYI, I just retested commit 6a093ae6e67e323165e3c7c08cb21b280e09092f by reducing the local_port range after squid is fully up. It shows source port reuse as well as the health checking service continues to...
> > Confirmed latest commit does not show reuse of outbound TCP source ports > > Please see my earlier [review](https://github.com/squid-cache/squid/pull/1115#discussion_r943464151) for that bug triage and suggested fixes. @rousskov I...