(trafficserver) branch master updated: [compress] Make warning if compress plugin has cache and range-request config (#11986)

[ma...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new d543c8a30e [compress] Make warning if compress plugin has cache and range-request config (#11986)
d543c8a30e is described below

commit d543c8a30e4447a422bf688cb0299cf3074ece51
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Tue Jan 28 08:10:51 2025 +0900

    [compress] Make warning if compress plugin has cache and range-request config (#11986)
---
 plugins/compress/configuration.cc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/plugins/compress/configuration.cc b/plugins/compress/configuration.cc
index 208ea79d8d..82cb15d75d 100644
--- a/plugins/compress/configuration.cc
+++ b/plugins/compress/configuration.cc
@@ -405,6 +405,11 @@ Configuration::Parse(const char *path)
   // Update the defaults for the last host configuration too, if needed.
   current_host_configuration->update_defaults();
 
+  // Check combination of configs
+  if (!current_host_configuration->cache() && current_host_configuration->range_request()) {
+    warning("Combination of 'cache false' and 'range-request true' might deliver corrupted content");
+  }
+
   if (state != kParseStart) {
     warning("the parser state indicates that data was expected when it reached the end of the file (%d)", state);
   }