Improve log output format

Also rename of field 'prefix' to 'api' in afb_ditf

Change-Id: Iaff49a3e829bca6e33023c888e95fb18fa0b85d8
Signed-off-by: José Bollo <jose.bollo@iot.bzh>

diff --git a/src/afb-ditf.c b/src/afb-ditf.c
index 4ee5af7..b5a5093 100644 (file)
--- a/src/afb-ditf.c
+++ b/src/afb-ditf.c
@@ -44,7 +44,7 @@ static void vverbose_cb(void *closure, int level, const char *file, int line, co
        if (vasprintf(&p, fmt, args) < 0)
                vverbose(level, file, line, function, fmt, args);
        else {
-               verbose(level, file, line, function, "%s {binding %s}", p, ditf->prefix);
+               verbose(level, file, line, function, "[API %s] %s", ditf->api, p);
                free(p);
        }
 }
@@ -61,10 +61,10 @@ static struct afb_event event_make_cb(void *closure, const char *name)
        struct afb_ditf *ditf = closure;
 
        /* makes the event name */
-       plen = strlen(ditf->prefix);
+       plen = strlen(ditf->api);
        nlen = strlen(name);
        event = alloca(nlen + plen + 2);
-       memcpy(event, ditf->prefix, plen);
+       memcpy(event, ditf->api, plen);
        event[plen] = '/';
        memcpy(event + plen + 1, name, nlen + 1);
 
@@ -79,10 +79,10 @@ static int event_broadcast_cb(void *closure, const char *name, struct json_objec
        struct afb_ditf *ditf = closure;
 
        /* makes the event name */
-       plen = strlen(ditf->prefix);
+       plen = strlen(ditf->api);
        nlen = strlen(name);
        event = alloca(nlen + plen + 2);
-       memcpy(event, ditf->prefix, plen);
+       memcpy(event, ditf->api, plen);
        event[plen] = '/';
        memcpy(event + plen + 1, name, nlen + 1);
 
@@ -205,33 +205,33 @@ static const struct afb_daemon_itf hooked_daemon_itf = {
        .queue_job = hooked_queue_job_cb
 };
 
-void afb_ditf_init_v2(struct afb_ditf *ditf, const char *prefix, struct afb_binding_data_v2 *data)
+void afb_ditf_init_v2(struct afb_ditf *ditf, const char *api, struct afb_binding_data_v2 *data)
 {
        ditf->version = 2;
        ditf->v2 = data;
        data->daemon.closure = ditf;
-       afb_ditf_rename(ditf, prefix);
+       afb_ditf_rename(ditf, api);
 }
 
-void afb_ditf_init_v1(struct afb_ditf *ditf, const char *prefix, struct afb_binding_interface_v1 *itf)
+void afb_ditf_init_v1(struct afb_ditf *ditf, const char *api, struct afb_binding_interface_v1 *itf)
 {
        ditf->version = 1;
        ditf->v1 = itf;
        itf->verbosity = verbosity;
        itf->mode = AFB_MODE_LOCAL;
        itf->daemon.closure = ditf;
-       afb_ditf_rename(ditf, prefix);
+       afb_ditf_rename(ditf, api);
 }
 
-void afb_ditf_rename(struct afb_ditf *ditf, const char *prefix)
+void afb_ditf_rename(struct afb_ditf *ditf, const char *api)
 {
-       ditf->prefix = prefix;
+       ditf->api = api;
        afb_ditf_update_hook(ditf);
 }
 
 void afb_ditf_update_hook(struct afb_ditf *ditf)
 {
-       int hooked = !!afb_hook_flags_ditf(ditf->prefix);
+       int hooked = !!afb_hook_flags_ditf(ditf->api);
        switch (ditf->version) {
        case 1:
                ditf->v1->daemon.itf = hooked ? &hooked_daemon_itf : &daemon_itf;

diff --git a/src/afb-ditf.h b/src/afb-ditf.h
index 6bc8498..5b8dbf5 100644 (file)
--- a/src/afb-ditf.h
+++ b/src/afb-ditf.h
@@ -23,15 +23,15 @@ struct afb_binding_data_v2;
 struct afb_ditf
 {
        int version;
-       const char *prefix;
+       const char *api;
        union {
                struct afb_binding_interface_v1 *v1;
                struct afb_binding_data_v2 *v2;
        };
 };
 
-extern void afb_ditf_init_v1(struct afb_ditf *ditf, const char *prefix, struct afb_binding_interface_v1 *itf);
-extern void afb_ditf_init_v2(struct afb_ditf *ditf, const char *prefix, struct afb_binding_data_v2 *data);
-extern void afb_ditf_rename(struct afb_ditf *ditf, const char *prefix);
+extern void afb_ditf_init_v1(struct afb_ditf *ditf, const char *api, struct afb_binding_interface_v1 *itf);
+extern void afb_ditf_init_v2(struct afb_ditf *ditf, const char *api, struct afb_binding_data_v2 *data);
+extern void afb_ditf_rename(struct afb_ditf *ditf, const char *api);
 extern void afb_ditf_update_hook(struct afb_ditf *ditf);
 

diff --git a/src/afb-hook.c b/src/afb-hook.c
index aa66b5d..6a11341 100644 (file)
--- a/src/afb-hook.c
+++ b/src/afb-hook.c
@@ -78,6 +78,18 @@ struct afb_hook_svc {
        void *closure; /**< closure for callbacks */
 };
 
+/**
+ * Definition of a hook for evt
+ */
+struct afb_hook_evt {
+       struct afb_hook_evt *next; /**< next hook */
+       unsigned refcount; /**< reference count */
+       char *pattern; /**< event pattern name hooked or NULL for any */
+       unsigned flags; /**< hook flags */
+       struct afb_hook_evt_itf *itf; /**< interface of hook */
+       void *closure; /**< closure for callbacks */
+};
+
 /* synchronisation across threads */
 static pthread_rwlock_t rwlock = PTHREAD_RWLOCK_INITIALIZER;
 
@@ -90,26 +102,55 @@ static struct afb_hook_ditf *list_of_ditf_hooks = NULL;
 /* list of hooks for svc */
 static struct afb_hook_svc *list_of_svc_hooks = NULL;
 
+/* list of hooks for evt */
+static struct afb_hook_evt *list_of_evt_hooks = NULL;
+
 /******************************************************************************
  * section: default callbacks for tracing requests
  *****************************************************************************/
 
+static char *_pbuf_(const char *fmt, va_list args, char **palloc, char *sbuf, size_t szsbuf)
+{
+       int rc;
+       va_list cp;
+
+       *palloc = NULL;
+       va_copy(cp, args);
+       rc = vsnprintf(sbuf, szsbuf, fmt, args);
+       if ((size_t)rc >= szsbuf) {
+               sbuf[szsbuf-1] = 0;
+               sbuf[szsbuf-2] = sbuf[szsbuf-3] = sbuf[szsbuf-4] = '.';
+               rc = vasprintf(palloc, fmt, cp);
+               if (rc >= 0)
+                       sbuf = *palloc;
+       }
+       va_end(cp);
+       return sbuf;
+}
+
+static void _hook_(const char *fmt1, const char *fmt2, va_list arg2, ...)
+{
+       char *tag, *data, *mem1, *mem2, buf1[256], buf2[2000];
+       va_list arg1;
+
+       data = _pbuf_(fmt2, arg2, &mem2, buf2, sizeof buf2);
+
+       va_start(arg1, arg2);
+       tag = _pbuf_(fmt1, arg1, &mem1, buf1, sizeof buf1);
+       va_end(arg1);
+
+       NOTICE("[HOOK %s] %s", tag, data);
+
+       free(mem1);
+       free(mem2);
+}
+
 static void _hook_xreq_(const struct afb_xreq *xreq, const char *format, ...)
 {
-       int len;
-       char *buffer;
        va_list ap;
-
        va_start(ap, format);
-       len = vasprintf(&buffer, format, ap);
+       _hook_("xreq-%06d:%s/%s", format, ap, xreq->hookindex, xreq->api, xreq->verb);
        va_end(ap);
-
-       if (len < 0)
-               NOTICE("hook xreq-%06d:%s/%s allocation error", xreq->hookindex, xreq->api, xreq->verb);
-       else {
-               NOTICE("hook xreq-%06d:%s/%s %s", xreq->hookindex, xreq->api, xreq->verb, buffer);
-               free(buffer);
-       }
 }
 
 static void hook_xreq_begin_default_cb(void * closure, const struct afb_xreq *xreq)
@@ -494,20 +535,10 @@ void afb_hook_unref_xreq(struct afb_hook_xreq *hook)
 
 static void _hook_ditf_(const struct afb_ditf *ditf, const char *format, ...)
 {
-       int len;
-       char *buffer;
        va_list ap;
-
        va_start(ap, format);
-       len = vasprintf(&buffer, format, ap);
+       _hook_("ditf-%s", format, ap, ditf->api);
        va_end(ap);
-
-       if (len < 0)
-               NOTICE("hook ditf-%s allocation error for %s", ditf->prefix, format);
-       else {
-               NOTICE("hook ditf-%s %s", ditf->prefix, buffer);
-               free(buffer);
-       }
 }
 
 static void hook_ditf_event_broadcast_before_cb(void *closure, const struct afb_ditf *ditf, const char *name, struct json_object *object)
@@ -613,7 +644,7 @@ static struct afb_hook_ditf_itf hook_ditf_default_itf = {
        while (hook) { \
                if (hook->itf->hook_ditf_##what \
                 && (hook->flags & afb_hook_flag_ditf_##what) != 0 \
-                && (!hook->api || !strcasecmp(hook->api, ditf->prefix))) { \
+                && (!hook->api || !strcasecmp(hook->api, ditf->api))) { \
                        hook->itf->hook_ditf_##what(hook->closure, __VA_ARGS__); \
                } \
                hook = hook->next; \
@@ -770,20 +801,10 @@ void afb_hook_unref_ditf(struct afb_hook_ditf *hook)
 
 static void _hook_svc_(const struct afb_svc *svc, const char *format, ...)
 {
-       int len;
-       char *buffer;
        va_list ap;
-
        va_start(ap, format);
-       len = vasprintf(&buffer, format, ap);
+       _hook_("svc-%s", format, ap, svc->api);
        va_end(ap);
-
-       if (len < 0)
-               NOTICE("hook svc-%s allocation error for %s", svc->api, format);
-       else {
-               NOTICE("hook svc-%s %s", svc->api, buffer);
-               free(buffer);
-       }
 }
 
 static void hook_svc_start_before_default_cb(void *closure, const struct afb_svc *svc)
@@ -983,86 +1004,16 @@ void afb_hook_unref_svc(struct afb_hook_svc *hook)
        }
 }
 
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-/*********************************************************
-* section hooking evt (event interface)
-*********************************************************/
-
-/**
- * Definition of a hook for evt
- */
-struct afb_hook_evt {
-       struct afb_hook_evt *next; /**< next hook */
-       unsigned refcount; /**< reference count */
-       char *pattern; /**< event pattern name hooked or NULL for any */
-       unsigned flags; /**< hook flags */
-       struct afb_hook_evt_itf *itf; /**< interface of hook */
-       void *closure; /**< closure for callbacks */
-};
-
-/* list of hooks for evt */
-static struct afb_hook_evt *list_of_evt_hooks = NULL;
-
-
 /******************************************************************************
  * section: default callbacks for tracing service interface (evt)
  *****************************************************************************/
 
 static void _hook_evt_(const char *evt, int id, const char *format, ...)
 {
-       int len;
-       char *buffer;
        va_list ap;
-
        va_start(ap, format);
-       len = vasprintf(&buffer, format, ap);
+       _hook_("evt-%s:%d", format, ap, evt, id);
        va_end(ap);
-
-       if (len < 0)
-               NOTICE("hook evt-%s:%d allocation error for %s", evt, id, format);
-       else {
-               NOTICE("hook evt-%s:%d %s", evt, id, buffer);
-               free(buffer);
-       }
 }
 
 static void hook_evt_create_default_cb(void *closure, const char *evt, int id)
@@ -1251,36 +1202,3 @@ void afb_hook_unref_evt(struct afb_hook_evt *hook)
                }
        }
 }
-
-#if 0
-#define afb_hook_flag_evt_create                       0x000001
-#define afb_hook_flag_evt_push_before                  0x000002
-#define afb_hook_flag_evt_push_after                   0x000004
-#define afb_hook_flag_evt_broadcast_before             0x000008
-#define afb_hook_flag_evt_broadcast_after              0x000010
-#define afb_hook_flag_evt_drop                         0x000020
-#define afb_hook_flag_evt_name                         0x000040
-
-struct afb_hook_evt_itf {
-       void (*hook_evt_create)(void *closure, const char *evt);
-       void (*hook_evt_push_before)(void *closure, const char *evt);
-       void (*hook_evt_push_after)(void *closure, const char *evt, int result);
-       void (*hook_evt_broadcast_before)(void *closure, const char *evt);
-       void (*hook_evt_broadcast_after)(void *closure, const char *evt, int result);
-       void (*hook_evt_drop)(void *closure, const char *evt);
-       void (*hook_evt_name)(void *closure, const char *evt);
-};
-
-extern void afb_hook_evt_create(const char *evt);
-extern void afb_hook_evt_push_before(const char *evt);
-extern int afb_hook_evt_push_after(const char *evt, int result);
-extern void afb_hook_evt_broadcast_before(const char *evt);
-extern int afb_hook_evt_broadcast_after(const char *evt, int result);
-extern void afb_hook_evt_drop(const char *evt);
-extern void afb_hook_evt_name(const char *evt);
-
-extern int afb_hook_flags_evt(const char *name);
-extern struct afb_hook_evt *afb_hook_create_evt(const char *name, int flags, struct afb_hook_evt_itf *itf, void *closure);
-extern struct afb_hook_evt *afb_hook_addref_evt(struct afb_hook_evt *hook);
-extern void afb_hook_unref_evt(struct afb_hook_evt *hook);
-#endif