Github user jpeach commented on a diff in the pull request:
https://github.com/apache/trafficserver/pull/971#discussion_r77854033
--- 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 --
I think the right check is to test whether there are any unregistered local
or config records. This can be done periodically since new records can be added
at config reload time. A records that is present in ``records.config`` will be
not be marked as registered intul a plugin calls ``TSMgmtStringCreate()`` or
the equivalent.
---
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.
---
