trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zw...@apache.org
Subject [trafficserver] branch 9.1.x updated: conf_remap: demote 'Invalid configuration' to warning (#7991)
Date Tue, 13 Jul 2021 20:25:18 GMT
This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.1.x by this push:
     new 99b1a3e  conf_remap: demote 'Invalid configuration' to warning (#7991)
99b1a3e is described below

commit 99b1a3e24c9424224f796334ab9bce9546e04405
Author: Randall Meyer <rrm@apache.org>
AuthorDate: Thu Jun 24 16:57:51 2021 -0700

    conf_remap: demote 'Invalid configuration' to warning (#7991)
    
    When I added this in b4241b99d0d6017397acf92a2562898b7b5e57b4, I
    miscategorized the log message. Since the plugin continues the loading,
    this should only be a warning.
    
    (cherry picked from commit 496134f331268ccdba6a0712617f777d08a3be12)
---
 plugins/conf_remap/conf_remap.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/plugins/conf_remap/conf_remap.cc b/plugins/conf_remap/conf_remap.cc
index 4a3d160..8cb9876 100644
--- a/plugins/conf_remap/conf_remap.cc
+++ b/plugins/conf_remap/conf_remap.cc
@@ -91,7 +91,7 @@ RemapConfigs::parse_inline(const char *arg)
   value = std::string(sep + 1, std::distance(sep + 1, arg + strlen(arg)));
 
   if (TSHttpTxnConfigFind(key.c_str(), -1 /* len */, &name, &type) != TS_SUCCESS)
{
-    TSError("[%s] Invalid configuration variable '%s'", PLUGIN_NAME, key.c_str());
+    TSWarning("[%s] Invalid configuration variable '%s'", PLUGIN_NAME, key.c_str());
     return true;
   }
 

Mime
View raw message