You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@trafficserver.apache.org by "maskit (via GitHub)" <gi...@apache.org> on 2023/06/20 19:54:06 UTC

[GitHub] [trafficserver] maskit commented on pull request #9869: Extract HttpUserAgent class from HttpSM

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: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org