ema commented on a change in pull request #6730:
URL: https://github.com/apache/trafficserver/pull/6730#discussion_r419900877



##########
File path: proxy/http/HttpTransact.cc
##########
@@ -7142,26 +7142,38 @@ 
HttpTransact::does_client_request_permit_storing(CacheControlResult *c, HTTPHdr
   return (true);
 }
 
+int
+HttpTransact::get_max_age(State *s, HTTPHdr *response)

Review comment:
       Good question, state is needed by TxnDebug. I have kept the two 
debugging statements (printing s_maxage or max_age depending on which one is 
defined) because they were used in calculate_document_freshness_limit. If we 
decide that they are not needed, after all, then we can avoid passing State as 
a parameter to the function. 




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to