Alex Rousskov

Results 212 comments of Alex Rousskov

``` testHttpRequest.cc:131:Assertion Test name: TestHttpRequest::testIPv6HostColonBug equality assertion failed - Expected: [2000:800::45] - Actual : 2000:800::45 ``` > I'm not 100% sure what these tests are trying to do The failing...

The core problems here are very different AFAICT. I fixed one of them in branch commit 6c9cb63, but that commit does not fix all the problems that should be fixed....

@yadij, the changes in this PR will start blocking some requests, including some requests that may have been allowed and "working" (at least to a certain extent, like fetching cache...

> Francesco: the PR is waiting for you to accept the review suggestions; have I missed anything? Yes, you have: * I am strongly against accepting those suggestions: https://github.com/squid-cache/squid/pull/1421#discussion_r1296146980 *...

> Hey, are we any closer to getting this bug fixed? Not since September 18, 2023. The PR is currently stuck. AFAICT, at least two things are required to make...

> I would like to get this fixed upsteam I assume that "this" is "bug 5154". I believe that [bug](https://bugs.squid-cache.org/show_bug.cgi?id=5154) has been fixed in master/v7 commit 97bbba6 a couple of...

> I have not wholly understood your comment yet (with a concrete image of codes), and it may take time Needless to say, please feel free to ask questions, especially...

Alleged facts: > [Alex](https://github.com/squid-cache/squid/pull/1668#issuecomment-2035419546): ERR_ZERO_SIZE_OBJECT error may only occur after active SslBump processing is finished. In other words, it should have no effect on SslBump code in general and the...

Please also double check the adjusted PR title/description. @jtstrs, have you seen this bug result in actual memory leaks with some common memory management library? I am just curious. It...