You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2021/07/13 20:25:08 UTC

[trafficserver] branch 9.0.x updated: conf_remap: demote 'Invalid configuration' to warning (#7991)

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

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


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

commit 1fdc297a55d36fac394d37587b42134c1f11bbb6
Author: Randall Meyer <rr...@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 34b6198..b1d7e08 100644
--- a/plugins/conf_remap/conf_remap.cc
+++ b/plugins/conf_remap/conf_remap.cc
@@ -89,7 +89,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;
   }