[trafficserver] 05/05: In conf_remap plugin, print name of non-existent or non-overriddable configuration variable in a file to error log.

[zw...@apache.org]

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

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

commit fe3c69e3d93b0bee12e7f64d9599a9b74084d963
Author: Walter Karas <wk...@oath.com>
AuthorDate: Tue Aug 7 18:27:11 2018 -0500

    In conf_remap plugin, print name of non-existent or non-overriddable configuration variable in a file to error log.
    
    (cherry picked from commit 342b3116731abb80134411e624c68b1c58cf9ed3)
---
 plugins/conf_remap/conf_remap.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/plugins/conf_remap/conf_remap.cc b/plugins/conf_remap/conf_remap.cc
index d5dc7f4..6a01772 100644
--- a/plugins/conf_remap/conf_remap.cc
+++ b/plugins/conf_remap/conf_remap.cc
@@ -173,7 +173,8 @@ RemapConfigs::parse_file(const char *filename)
     // Find the configuration name
     tok = strtok_r(nullptr, " \t", &ln);
     if (TSHttpTxnConfigFind(tok, -1, &name, &expected_type) != TS_SUCCESS) {
-      TSError("[%s] File %s, line %d: no records.config name given", PLUGIN_NAME, path.c_str(), line_num);
+      TSError("[%s] File %s, line %d: %s is not a configuration variable or cannot be overridden", PLUGIN_NAME, path.c_str(),
+              line_num, tok);
       continue;
     }