aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJosé Bollo <jose.bollo@iot.bzh>2017-06-01 13:19:16 +0200
committerJosé Bollo <jose.bollo@iot.bzh>2017-06-01 13:19:16 +0200
commit6a325d7f101c986e7329ef912c0458719f95c096 (patch)
tree53f35c58db9bd4d1906a2cfee8e3708f4c783607
parentb67e18b39830a01750721787bf3bdc5d71983144 (diff)
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>
-rw-r--r--src/afb-ditf.c24
-rw-r--r--src/afb-ditf.h8
-rw-r--r--src/afb-hook.c194
3 files changed, 72 insertions, 154 deletions
diff --git a/src/afb-ditf.c b/src/afb-ditf.c
index 4ee5af7f..b5a50930 100644
--- 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 6bc84986..5b8dbf5a 100644
--- 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 aa66b5de..6a11341c 100644
--- 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