Github user SolidWallOfCode commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/971#discussion_r77874593
--- Diff: proxy/Main.cc ---
@@ -476,6 +476,18 @@ check_config_directories(void)
}
}
+namespace {
+ // Values with names that have this prefix are exempted from being
considered "unrecognized".
+ const char PLUGIN_CONFIG_RECORD_PREFIX[] = "proxy.config.plugin.";
--- End diff --
This is the root of the problem in that if arbitrary config names can be
used by plugins, you can't do validity checks. The consensus at the bug scrub
was it was reasonable to restrict plugin configuration values to that subtree.
I could move the check after plugin initialization so that if a plugin does
call {{TSMgmtStringCreate}} it doesn't generate a warning.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
