JakeChampion commented on code in PR #12618:
URL: https://github.com/apache/trafficserver/pull/12618#discussion_r2470214663
##########
tests/gold_tests/headers/replays/normalized_ae_varied_transactions.replay.yaml:
##########
@@ -190,14 +209,23 @@ sessions:
delay: 100ms
server-response:
- <<: *404_response
+ status: 200
+ reason: OK
+ headers:
+ fields:
+ - [ Transfer-Encoding, chunked ]
+ - [ Cache-Control, max-age=300 ]
+ - [ Content-Encoding, br ]
+ - [ Vary, Accept-Encoding ]
+ - [ Connection, close ]
+ - [ X-Response-Identifier, Br-Accept-Encoding ]
proxy-response:
status: 200
headers:
fields:
- - [ X-Response-Identifier, { value: Gzip-Accept-Encoding, as: equal } ]
- - [ X-Cache, { value: hit-fresh, as: equal } ]
+ - [ X-Response-Identifier, { value: Br-Accept-Encoding, as: equal } ]
+ - [ X-Cache, { value: miss, as: equal } ]
Review Comment:
I believe this is because this origin’s is configured with normalize_ae set
to 0 which keeps the Accept-Encoding header verbatim, no normalisation - and
the accept-encoding header value is different to the previously seen values,
which is why this is a cache miss
--
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]
