[TS-3244] stats_over_http plugin does not get the optional path argument properly

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 5.2.0, 5.3.0
• Component/s: Plugins
• Assignee: Leif Hedstrom
• Reporter: Leif Hedstrom
• Created: 17/Dec/14 20:37
• Updated: 18/Dec/14 05:06
• Resolved: 17/Dec/14 22:58

Description

To override the default path for stats_over_http.so with your own, you should add this to plugin.config:

stats_over_http.so /my_stats_path

This no longer works. Patch would be

diff --git a/plugins/stats_over_http/stats_over_http.c b/plugins/stats_over_http/stats_over_http.c
index 720a7e7..9c4f8bf 100644
--- a/plugins/stats_over_http/stats_over_http.c
+++ b/plugins/stats_over_http/stats_over_http.c
@@ -312,8 +312,8 @@ init:
   argc -= optind;
   argv += optind;
 
-  if (argc > 1) {
-    url_path = TSstrdup(argv[1] + ('/' == argv[1][0] ? 1 : 0)); /* Skip leading / */
+  if (argc > 0) {
+    url_path = TSstrdup(argv[0] + ('/' == argv[0][0] ? 1 : 0)); /* Skip leading / */
   }
   url_path_len = strlen(url_path);