Alex Rousskov
Alex Rousskov
I also adjusted PR title to be a bit more specific. Please polish further as needed, of course.
> I've got a bug for this but it's not been looked at since I last pointed out the tags weren't created Bugzilla has failed to notify me about bug...
> When this Parser class was designed and implemented there was an assumption that HTTP/2+ being binary would need an entirely different set of Parser internals, thus a different Parser...
> HTTP/1 and HTTP/2 syntax and use of TCP are still highly intertwined. Regardless of the accuracy of that assertion (which I hope we do not need to perfect and...
TLDR: ```C++ // Amos proposal (as far as I can tell; with an empty One::Parser I added) namespace Http { /// The API to fetch HTTP semantic details. class Parser...
> You came close, however the "headers parser" is a separate "class HttpHeader" in the current code and not touched by this PR. I think this is just a minor...
> It is unclear to me why this feature addition is tied to intercept+SSL-Bump usage. The PROXY protocol bytes occur before any TLS bytes and may be used with non-intercepted...
> Amos: see PR #873 for separate implementation receiving/parsing PROXY protocol. It will leave you with the ::Server::xaction->pp2Client object ::Server::xaction should not exist (as discussed in https://github.com/squid-cache/squid/pull/865#discussion_r674066436) so any PR...
> the state of this PR right now is that we need to complete the issues in PR #835, rebase this on the resulting master, then see if this PR...
I have removed the `S-waiting-for-author` label: This PR is not waiting for its author. Since June 2021, this PR is waiting for PR #835 (which was promptly created upon @yadij...