You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "Nick Kew (JIRA)" <ji...@apache.org> on 2013/07/16 01:50:48 UTC
[jira] [Comment Edited] (TS-2036) Management update API for plugins
doesn't work
[ https://issues.apache.org/jira/browse/TS-2036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13709194#comment-13709194 ]
Nick Kew edited comment on TS-2036 at 7/15/13 11:49 PM:
--------------------------------------------------------
Aaargh! "Provide patch" option just marks it as PatchAvailable, without offering the option to provide a patch.
git push is also eluding me with "error: no DAV locking support on https://git-wip-us.apache.org/repos/asf/trafficserver.git/", so here's the patch:
{code}
-#ifndef TS_NO_API
- plugin_init(system_config_directory); // plugin.config
-#else
+#ifdef TS_NO_API
api_init(); // we still need to initialize some of the data
// i.e. http_global_hooks
+#else
+ plugin_init(system_config_directory); // plugin.config
pmgmt->registerPluginCallbacks(global_config_cbs);
#endif
{code}
Summary: plugin callbacks need to be registered when there is an API, not when there isn't! The patch switches #ifndef to #ifdef, because confusion over the double-negative is probably what caused the bug in the first place.
was (Author: nick@webthing.com):
Aaargh! "Provide patch" option just marks it as PatchAvailable, without offering the option to provide a patch.
git push is also eluding me with "error: no DAV locking support on https://git-wip-us.apache.org/repos/asf/trafficserver.git/", so here's the patch:
-#ifndef TS_NO_API
- plugin_init(system_config_directory); // plugin.config
-#else
+#ifdef TS_NO_API
api_init(); // we still need to initialize some of the data
// i.e. http_global_hooks
+#else
+ plugin_init(system_config_directory); // plugin.config
pmgmt->registerPluginCallbacks(global_config_cbs);
#endif
Summary: plugin callbacks need to be registered when there is an API, not when there isn't! The patch switches #ifndef to #ifdef, because confusion over the double-negative is probably what caused the bug in the first place.
> Management update API for plugins doesn't work
> ----------------------------------------------
>
> Key: TS-2036
> URL: https://issues.apache.org/jira/browse/TS-2036
> Project: Traffic Server
> Issue Type: Bug
> Components: Management API
> Reporter: Nick Kew
> Assignee: Nick Kew
>
> Functions registered with TSMgmtUpdateRegister are never called.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira