+1 -Bryan
> On Aug 11, 2020, at 9:59 PM, Masakazu Kitajo <mas...@apache.org> wrote: > > I'd like to propose deprecating "cqhv" log field on 9.0.0. > > The documentation says "cqhv" logs "Client request HTTP version", however, > it's been false for a long time because the value never be HTTP/2. What it > actually logs is "HTTP version that ATS internally uses". As some of you may > know, ATS processes HTTP/2 requests by converting the requests to HTTP/1.1 > internally. And "cqhv" logs the HTTP version after the conversion. So it's > always HTTP/1.1 or maybe HTTP/1.0, and I guess most people don't care about > the internal representation. > > And we have another tag field "cqpv" that logs "Client request protocol and > version", which supports HTTP/2 and also HTTP/3. The value matches with the > protocol and version you can see on the client side. This is probably what > you really want to know. > > I'm not sure whether the current behavior of "cqhv" is intentional or not. > Maybe we should have fixed it before we introduce "cqpv". Even so, I think we > can deprecate "cqhv" since we already have "cqpv". > > I'm fine with keeping "cqhv" if some of you need it, but we should probably > update the documentation in that case. > > Thanks, > Masakazu
