bryancall commented on code in PR #12946: URL: https://github.com/apache/trafficserver/pull/12946#discussion_r2898418554
########## doc/admin-guide/plugins/compress.en.rst: ########## @@ -98,10 +99,21 @@ Per site configuration for remap plugin should be ignored. cache ----- -When set to ``true``, causes |TS| to cache both the compressed and uncompressed -versions of the content as :term:`alternates <alternate>`. When set to -``false``, |TS| will cache only the compressed or decompressed variant returned -by the origin. Enabled by default. +Controls which version of the response is stored in cache when the compress +transform runs (i.e., when the client sends ``Accept-Encoding``). + +When set to ``true``, the compressed (transformed) response is cached. When set +to ``false``, the uncompressed (untransformed) response is cached and +compression is performed on-the-fly for subsequent cache hits. + +.. note:: + + The plugin always adds ``Vary: Accept-Encoding`` to compressible responses. + This causes |TS| to store compressed and uncompressed responses as separate + :term:`alternates <alternate>`, so both variants will be cached after clients + of each type have made a request — regardless of this setting. Review Comment: Fixed in a9ce600 — reworded the note to clarify that `Vary: Accept-Encoding` creates separate cache keys, but which body representation is actually stored depends on the `cache` option. -- 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]
