JosiahWI commented on PR #12618:
URL: https://github.com/apache/trafficserver/pull/12618#issuecomment-3462367549

   @JakeChampion You are right, I had missed that ATS cleared the 
`Accept-Encoding: ` field in some cases with `normalize_ae` set to a nonzero 
value. In those cases, it is correct to match the previous `Accept-Encoding: ` 
response, as you said.
   
   Assuming that `normalize_ae` does only affect the request to origin and not 
which alternate is served from cache, is it correct to serve the cached 
document after getting the 404 from the origin? My intuition says that the 404 
should replace the existing 200 in cache, but I haven't checked the RFC, so I 
may be wrong about that.
   
   Ideally, I think it would be good to serve directly from cache and skip the 
request to origin (unless the response is stale), as you suggested. (In other 
words, the `Accept-Encoding` header _should_ be normalized before looking for 
alternates, not after).


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

Reply via email to