From: Mathieu STOFFEL <mathieu.stoffel@atos.net>
Date: Mon, 27 Sep 2021 14:37:05 +0000 (+0200)
Subject: Feature-redfish: enabling libredfish logging on collectd debug mode (#3907)
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=69aabee7ccffa486218c327f7ebf3a6cdbe43851;p=thirdparty%2Fcollectd.git

Feature-redfish: enabling libredfish logging on collectd debug mode (#3907)

Changelog: enabling libredfish logging on collectd debug mode

In order for `libredfish` to log internal information, which are more
than useful to debugging purposes, a printing function has to be
registered through a hook : a global function pointer named `gDebugFunc`.

The `plugin_log` function is registered as the logging function through
the aforementioned hook by the `redfish_init` function, when the
`--enable-debug` option is specified to the `configure` script.
---

diff --git a/src/redfish.c b/src/redfish.c
index 947fb392e..8cfbb2924 100644
--- a/src/redfish.c
+++ b/src/redfish.c
@@ -113,6 +113,10 @@ static int redfish_validate_config(void);
 static void *redfish_worker_thread(void __attribute__((unused)) * args);
 
 #if COLLECT_DEBUG
+/* Hook exposed by the libredfish library to define a printing function
+ * dedicated to logging purposes: */
+extern libRedfishDebugFunc gDebugFunc;
+
 static void redfish_print_config(void) {
   DEBUG(PLUGIN_NAME ": ====================CONFIGURATION====================");
   DEBUG(PLUGIN_NAME ": SERVICES: %d", llist_size(ctx.services));
@@ -193,6 +197,10 @@ static void redfish_job_destroy(redfish_job_t *job) {
 
 static int redfish_init(void) {
 #if COLLECT_DEBUG
+  /* Registering plugin_log as the printing function dedicated to logging
+   * purposes within libredfish: */
+  gDebugFunc = plugin_log;
+
   redfish_print_config();
 #endif
   int ret = redfish_validate_config();