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;
}