maskit commented on PR #9869: URL: https://github.com/apache/trafficserver/pull/9869#issuecomment-1599413737
Hmm, the goal is still unclear to me. If it's going to be like we can get anything about a user agent from `HttpUserAgent`, and anything about a server from `HttpServer` that kinda make sense. But we already have `client_info` and `server_info` in `HttpTransact::State`. What would be the difference? And why don't you move other variables that are related to client (e.g. `client_request_hdr_bytes`, `client_tcp_reused`, client_protocol`, etc) into the class? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
