X-Git-Url: https://git.octo.it/?a=blobdiff_plain;f=src%2Fcurl_json.c;h=a84cba042b924d6e3b7667066889ead4ab41ca72;hb=103f05e098865196fc5f28df51e99b64fd6b5202;hp=24e1df1e9b016d5086b6efdae445fdc6e9e6b625;hpb=56c7b21899e44b57b5031a21ca3652b30f4fc1aa;p=collectd.git diff --git a/src/curl_json.c b/src/curl_json.c index 24e1df1e..a84cba04 100644 --- a/src/curl_json.c +++ b/src/curl_json.c @@ -1,7 +1,7 @@ /** * collectd - src/curl_json.c * Copyright (C) 2009 Doug MacEachern - * Copyright (C) 2006-2011 Florian octo Forster + * Copyright (C) 2006-2013 Florian octo Forster * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License as published by the @@ -28,7 +28,12 @@ #include "utils_avltree.h" #include "utils_complain.h" +#include +#include +#include + #include + #include #if HAVE_YAJL_YAJL_VERSION_H # include @@ -40,7 +45,7 @@ #define CJ_DEFAULT_HOST "localhost" #define CJ_KEY_MAGIC 0x43484b59UL /* CHKY */ -#define CJ_IS_KEY(key) (key)->magic == CJ_KEY_MAGIC +#define CJ_IS_KEY(key) ((key)->magic == CJ_KEY_MAGIC) #define CJ_ANY "*" #define COUCH_MIN(x,y) ((x) < (y) ? (x) : (y)) @@ -48,10 +53,10 @@ struct cj_key_s; typedef struct cj_key_s cj_key_t; struct cj_key_s /* {{{ */ { + unsigned long magic; char *path; char *type; char *instance; - unsigned long magic; }; /* }}} */ @@ -60,13 +65,19 @@ struct cj_s /* {{{ */ char *instance; char *host; + char *sock; + char *url; char *user; char *pass; char *credentials; - int verify_peer; - int verify_host; + _Bool digest; + _Bool verify_peer; + _Bool verify_host; char *cacert; + struct curl_slist *headers; + char *post_body; + cdtime_t interval; CURL *curl; char curl_errbuf[CURL_ERROR_SIZE]; @@ -80,6 +91,8 @@ struct cj_s /* {{{ */ c_avl_tree_t *tree; cj_key_t *key; }; + _Bool in_array; + int index; char name[DATA_MAX_NAME_LEN]; } state[YAJL_MAX_DEPTH]; }; @@ -92,7 +105,6 @@ typedef unsigned int yajl_len_t; #endif static int cj_read (user_data_t *ud); -static int cj_curl_perform (cj_t *db, CURL *curl); static void cj_submit (cj_t *db, cj_key_t *key, value_t *value); static size_t cj_curl_callback (void *buf, /* {{{ */ @@ -122,7 +134,8 @@ static size_t cj_curl_callback (void *buf, /* {{{ */ if (status != yajl_status_ok) { unsigned char *msg = - yajl_get_error(db->yajl, 1, (unsigned char *)buf, len); + yajl_get_error(db->yajl, /* verbose = */ 1, + /* jsonText = */ (unsigned char *) buf, (unsigned int) len); ERROR ("curl_json plugin: yajl_parse failed: %s", msg); yajl_free_error(db->yajl, msg); return (0); /* abort write callback */ @@ -164,12 +177,44 @@ static int cj_get_type (cj_key_t *key) return ds->ds[0].type; } +static int cj_cb_map_key (void *ctx, const unsigned char *val, + yajl_len_t len); + +static void cj_cb_inc_array_index (void *ctx, _Bool update_key) +{ + cj_t *db = (cj_t *)ctx; + + if (!db->state[db->depth].in_array) + return; + + db->state[db->depth].index++; + + if (update_key) + { + char name[DATA_MAX_NAME_LEN]; + + ssnprintf (name, sizeof (name), "%d", db->state[db->depth].index - 1); + + cj_cb_map_key (ctx, (unsigned char *)name, (yajl_len_t) strlen (name)); + } +} + /* yajl callbacks */ #define CJ_CB_ABORT 0 #define CJ_CB_CONTINUE 1 -/* "number" may not be null terminated, so copy it into a buffer before - * parsing. */ +static int cj_cb_boolean (void * ctx, int boolVal) +{ + cj_cb_inc_array_index (ctx, /* update_key = */ 0); + return (CJ_CB_CONTINUE); +} + +static int cj_cb_null (void * ctx) +{ + cj_cb_inc_array_index (ctx, /* update_key = */ 0); + return (CJ_CB_CONTINUE); +} + static int cj_cb_number (void *ctx, const char *number, yajl_len_t number_len) { @@ -177,41 +222,29 @@ static int cj_cb_number (void *ctx, cj_t *db = (cj_t *)ctx; cj_key_t *key = db->state[db->depth].key; - char *endptr; value_t vt; int type; + int status; - if (key == NULL) - return (CJ_CB_CONTINUE); - + /* Create a null-terminated version of the string. */ memcpy (buffer, number, number_len); buffer[sizeof (buffer) - 1] = 0; - type = cj_get_type (key); - if (type < 0) + if ((key == NULL) || !CJ_IS_KEY (key)) { + if (key != NULL) + NOTICE ("curl_json plugin: Found \"%s\", but the configuration expects" + " a map.", buffer); + cj_cb_inc_array_index (ctx, /* update_key = */ 0); return (CJ_CB_CONTINUE); - - endptr = NULL; - errno = 0; - - if (type == DS_TYPE_COUNTER) - vt.counter = (counter_t) strtoull (buffer, &endptr, /* base = */ 0); - else if (type == DS_TYPE_GAUGE) - vt.gauge = (gauge_t) strtod (buffer, &endptr); - else if (type == DS_TYPE_DERIVE) - vt.derive = (derive_t) strtoll (buffer, &endptr, /* base = */ 0); - else if (type == DS_TYPE_ABSOLUTE) - vt.absolute = (absolute_t) strtoull (buffer, &endptr, /* base = */ 0); - else - { - ERROR ("curl_json plugin: Unknown data source type: \"%s\"", key->type); - return (CJ_CB_ABORT); + } else { + cj_cb_inc_array_index (ctx, /* update_key = */ 1); } - if ((endptr == &buffer[0]) || (errno != 0)) + type = cj_get_type (key); + status = parse_value (buffer, &vt, type); + if (status != 0) { - NOTICE ("curl_json plugin: Overflow while parsing number. " - "Ignoring this value."); + NOTICE ("curl_json plugin: Unable to parse number: \"%s\"", buffer); return (CJ_CB_CONTINUE); } @@ -219,8 +252,11 @@ static int cj_cb_number (void *ctx, return (CJ_CB_CONTINUE); } /* int cj_cb_number */ -static int cj_cb_map_key (void *ctx, const unsigned char *val, - yajl_len_t len) +/* Queries the key-tree of the parent context for "in_name" and, if found, + * updates the "key" field of the current context. Otherwise, "key" is set to + * NULL. */ +static int cj_cb_map_key (void *ctx, + unsigned char const *in_name, yajl_len_t in_name_len) { cj_t *db = (cj_t *)ctx; c_avl_tree_t *tree; @@ -229,12 +265,16 @@ static int cj_cb_map_key (void *ctx, const unsigned char *val, if (tree != NULL) { - cj_key_t *value; + cj_key_t *value = NULL; char *name; + size_t name_len; + /* Create a null-terminated version of the name. */ name = db->state[db->depth].name; - len = COUCH_MIN(len, sizeof (db->state[db->depth].name)-1); - sstrncpy (name, (char *)val, len+1); + name_len = COUCH_MIN ((size_t) in_name_len, + sizeof (db->state[db->depth].name) - 1); + memcpy (name, in_name, name_len); + name[name_len] = 0; if (c_avl_get (tree, name, (void *) &value) == 0) db->state[db->depth].key = value; @@ -250,42 +290,17 @@ static int cj_cb_map_key (void *ctx, const unsigned char *val, static int cj_cb_string (void *ctx, const unsigned char *val, yajl_len_t len) { - cj_t *db = (cj_t *)ctx; - c_avl_tree_t *tree; - char *ptr; - - if (db->depth != 1) /* e.g. _all_dbs */ - return (CJ_CB_CONTINUE); - - cj_cb_map_key (ctx, val, len); /* same logic */ - - tree = db->state[db->depth].tree; - - if ((tree != NULL) && (ptr = rindex (db->url, '/'))) - { - char url[PATH_MAX]; - CURL *curl; - - /* url =~ s,[^/]+$,$name, */ - len = (ptr - db->url) + 1; - ptr = url; - sstrncpy (ptr, db->url, sizeof (url)); - sstrncpy (ptr + len, db->state[db->depth].name, sizeof (url) - len); - - curl = curl_easy_duphandle (db->curl); - curl_easy_setopt (curl, CURLOPT_URL, url); - cj_curl_perform (db, curl); - curl_easy_cleanup (curl); - } - return (CJ_CB_CONTINUE); -} + /* Handle the string as if it was a number. */ + return (cj_cb_number (ctx, (const char *) val, len)); +} /* int cj_cb_string */ static int cj_cb_start (void *ctx) { cj_t *db = (cj_t *)ctx; if (++db->depth >= YAJL_MAX_DEPTH) { - ERROR ("curl_json plugin: %s depth exceeds max, aborting.", db->url); + ERROR ("curl_json plugin: %s depth exceeds max, aborting.", + db->url ? db->url : db->sock); return (CJ_CB_ABORT); } return (CJ_CB_CONTINUE); @@ -301,6 +316,7 @@ static int cj_cb_end (void *ctx) static int cj_cb_start_map (void *ctx) { + cj_cb_inc_array_index (ctx, /* update_key = */ 1); return cj_cb_start (ctx); } @@ -311,17 +327,25 @@ static int cj_cb_end_map (void *ctx) static int cj_cb_start_array (void * ctx) { + cj_t *db = (cj_t *)ctx; + cj_cb_inc_array_index (ctx, /* update_key = */ 1); + if (db->depth+1 < YAJL_MAX_DEPTH) { + db->state[db->depth+1].in_array = 1; + db->state[db->depth+1].index = 0; + } return cj_cb_start (ctx); } static int cj_cb_end_array (void * ctx) { + cj_t *db = (cj_t *)ctx; + db->state[db->depth].in_array = 0; return cj_cb_end (ctx); } static yajl_callbacks ycallbacks = { - NULL, /* null */ - NULL, /* boolean */ + cj_cb_null, /* null */ + cj_cb_boolean, /* boolean */ NULL, /* integer */ NULL, /* double */ cj_cb_number, @@ -389,19 +413,28 @@ static void cj_free (void *arg) /* {{{ */ sfree (db->instance); sfree (db->host); + sfree (db->sock); + sfree (db->url); sfree (db->user); sfree (db->pass); sfree (db->credentials); sfree (db->cacert); + sfree (db->post_body); + curl_slist_free_all (db->headers); sfree (db); } /* }}} void cj_free */ /* Configuration handling functions {{{ */ -static int cj_config_add_string (const char *name, char **dest, /* {{{ */ - oconfig_item_t *ci) +static c_avl_tree_t *cj_avl_create(void) +{ + return c_avl_create ((int (*) (const void *, const void *)) strcmp); +} + +static int cj_config_append_string (const char *name, struct curl_slist **dest, /* {{{ */ + oconfig_item_t *ci) { if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_STRING)) { @@ -409,32 +442,12 @@ static int cj_config_add_string (const char *name, char **dest, /* {{{ */ return (-1); } - sfree (*dest); - *dest = strdup (ci->values[0].value.string); + *dest = curl_slist_append(*dest, ci->values[0].value.string); if (*dest == NULL) return (-1); return (0); -} /* }}} int cj_config_add_string */ - -static int cj_config_set_boolean (const char *name, int *dest, /* {{{ */ - oconfig_item_t *ci) -{ - if ((ci->values_num != 1) || (ci->values[0].type != OCONFIG_TYPE_BOOLEAN)) - { - WARNING ("curl_json plugin: `%s' needs exactly one boolean argument.", name); - return (-1); - } - - *dest = ci->values[0].value.boolean ? 1 : 0; - - return (0); -} /* }}} int cj_config_set_boolean */ - -static c_avl_tree_t *cj_avl_create(void) -{ - return c_avl_create ((int (*) (const void *, const void *)) strcmp); -} +} /* }}} int cj_config_append_string */ static int cj_config_add_key (cj_t *db, /* {{{ */ oconfig_item_t *ci) @@ -462,7 +475,7 @@ static int cj_config_add_key (cj_t *db, /* {{{ */ if (strcasecmp ("Key", ci->key) == 0) { - status = cj_config_add_string ("Key", &key->path, ci); + status = cf_util_get_string (ci, &key->path); if (status != 0) { sfree (key); @@ -482,9 +495,9 @@ static int cj_config_add_key (cj_t *db, /* {{{ */ oconfig_item_t *child = ci->children + i; if (strcasecmp ("Type", child->key) == 0) - status = cj_config_add_string ("Type", &key->type, child); + status = cf_util_get_string (child, &key->type); else if (strcasecmp ("Instance", child->key) == 0) - status = cj_config_add_string ("Instance", &key->instance, child); + status = cf_util_get_string (child, &key->instance); else { WARNING ("curl_json plugin: Option `%s' not allowed here.", child->key); @@ -538,6 +551,7 @@ static int cj_config_add_key (cj_t *db, /* {{{ */ len = ptr-name; if (len == 0) break; + len = COUCH_MIN(len, sizeof (ent)-1); sstrncpy (ent, name, len+1); if (c_avl_get (tree, ent, (void *) &value) != 0) @@ -575,8 +589,7 @@ static int cj_init_curl (cj_t *db) /* {{{ */ curl_easy_setopt (db->curl, CURLOPT_NOSIGNAL, 1L); curl_easy_setopt (db->curl, CURLOPT_WRITEFUNCTION, cj_curl_callback); curl_easy_setopt (db->curl, CURLOPT_WRITEDATA, db); - curl_easy_setopt (db->curl, CURLOPT_USERAGENT, - PACKAGE_NAME"/"PACKAGE_VERSION); + curl_easy_setopt (db->curl, CURLOPT_USERAGENT, COLLECTD_USERAGENT); curl_easy_setopt (db->curl, CURLOPT_ERRORBUFFER, db->curl_errbuf); curl_easy_setopt (db->curl, CURLOPT_URL, db->url); @@ -598,6 +611,13 @@ static int cj_init_curl (cj_t *db) /* {{{ */ ssnprintf (db->credentials, credentials_size, "%s:%s", db->user, (db->pass == NULL) ? "" : db->pass); curl_easy_setopt (db->curl, CURLOPT_USERPWD, db->credentials); + + if (db->digest) + { + curl_easy_setopt (db->curl, CURLOPT_HTTPAUTH, CURLAUTH_DIGEST); + curl_easy_setopt (db->curl, CURLOPT_USERNAME, db->user); + curl_easy_setopt (db->curl, CURLOPT_PASSWORD, db->pass); + } } curl_easy_setopt (db->curl, CURLOPT_SSL_VERIFYPEER, (long) db->verify_peer); @@ -605,6 +625,10 @@ static int cj_init_curl (cj_t *db) /* {{{ */ db->verify_host ? 2L : 0L); if (db->cacert != NULL) curl_easy_setopt (db->curl, CURLOPT_CAINFO, db->cacert); + if (db->headers != NULL) + curl_easy_setopt (db->curl, CURLOPT_HTTPHEADER, db->headers); + if (db->post_body != NULL) + curl_easy_setopt (db->curl, CURLOPT_POSTFIELDS, db->post_body); return (0); } /* }}} int cj_init_curl */ @@ -632,20 +656,20 @@ static int cj_config_add_url (oconfig_item_t *ci) /* {{{ */ memset (db, 0, sizeof (*db)); if (strcasecmp ("URL", ci->key) == 0) - { - status = cj_config_add_string ("URL", &db->url, ci); - if (status != 0) - { - sfree (db); - return (status); - } - } + status = cf_util_get_string (ci, &db->url); + else if (strcasecmp ("Sock", ci->key) == 0) + status = cf_util_get_string (ci, &db->sock); else { ERROR ("curl_json plugin: cj_config: " "Invalid key: %s", ci->key); return (-1); } + if (status != 0) + { + sfree (db); + return (status); + } /* Fill the `cj_t' structure.. */ for (i = 0; i < ci->children_num; i++) @@ -653,21 +677,29 @@ static int cj_config_add_url (oconfig_item_t *ci) /* {{{ */ oconfig_item_t *child = ci->children + i; if (strcasecmp ("Instance", child->key) == 0) - status = cj_config_add_string ("Instance", &db->instance, child); + status = cf_util_get_string (child, &db->instance); else if (strcasecmp ("Host", child->key) == 0) - status = cj_config_add_string ("Host", &db->host, child); - else if (strcasecmp ("User", child->key) == 0) - status = cj_config_add_string ("User", &db->user, child); - else if (strcasecmp ("Password", child->key) == 0) - status = cj_config_add_string ("Password", &db->pass, child); - else if (strcasecmp ("VerifyPeer", child->key) == 0) - status = cj_config_set_boolean ("VerifyPeer", &db->verify_peer, child); - else if (strcasecmp ("VerifyHost", child->key) == 0) - status = cj_config_set_boolean ("VerifyHost", &db->verify_host, child); - else if (strcasecmp ("CACert", child->key) == 0) - status = cj_config_add_string ("CACert", &db->cacert, child); + status = cf_util_get_string (child, &db->host); + else if (db->url && strcasecmp ("User", child->key) == 0) + status = cf_util_get_string (child, &db->user); + else if (db->url && strcasecmp ("Password", child->key) == 0) + status = cf_util_get_string (child, &db->pass); + else if (strcasecmp ("Digest", child->key) == 0) + status = cf_util_get_boolean (child, &db->digest); + else if (db->url && strcasecmp ("VerifyPeer", child->key) == 0) + status = cf_util_get_boolean (child, &db->verify_peer); + else if (db->url && strcasecmp ("VerifyHost", child->key) == 0) + status = cf_util_get_boolean (child, &db->verify_host); + else if (db->url && strcasecmp ("CACert", child->key) == 0) + status = cf_util_get_string (child, &db->cacert); + else if (db->url && strcasecmp ("Header", child->key) == 0) + status = cj_config_append_string ("Header", &db->headers, child); + else if (db->url && strcasecmp ("Post", child->key) == 0) + status = cf_util_get_string (child, &db->post_body); else if (strcasecmp ("Key", child->key) == 0) status = cj_config_add_key (db, child); + else if (strcasecmp ("Interval", child->key) == 0) + status = cf_util_get_cdtime(child, &db->interval); else { WARNING ("curl_json plugin: Option `%s' not allowed here.", child->key); @@ -682,11 +714,11 @@ static int cj_config_add_url (oconfig_item_t *ci) /* {{{ */ { if (db->tree == NULL) { - WARNING ("curl_json plugin: No (valid) `Key' block " - "within `URL' block `%s'.", db->url); + WARNING ("curl_json plugin: No (valid) `Key' block within `%s' \"`%s'\".", + db->url ? "URL" : "Sock", db->url ? db->url : db->sock); status = -1; } - if (status == 0) + if (status == 0 && db->url) status = cj_init_curl (db); } @@ -694,7 +726,10 @@ static int cj_config_add_url (oconfig_item_t *ci) /* {{{ */ if (status == 0) { user_data_t ud; - char cb_name[DATA_MAX_NAME_LEN]; + char *cb_name; + struct timespec interval = { 0, 0 }; + + CDTIME_T_TO_TIMESPEC (db->interval, &interval); if (db->instance == NULL) db->instance = strdup("default"); @@ -706,11 +741,13 @@ static int cj_config_add_url (oconfig_item_t *ci) /* {{{ */ ud.data = (void *) db; ud.free_func = cj_free; - ssnprintf (cb_name, sizeof (cb_name), "curl_json-%s-%s", - db->instance, db->url); + cb_name = ssnprintf_alloc ("curl_json-%s-%s", + db->instance, db->url ? db->url : db->sock); plugin_register_complex_read (/* group = */ NULL, cb_name, cj_read, - /* interval = */ NULL, &ud); + /* interval = */ (db->interval > 0) ? &interval : NULL, + &ud); + sfree (cb_name); } else { @@ -736,7 +773,8 @@ static int cj_config (oconfig_item_t *ci) /* {{{ */ { oconfig_item_t *child = ci->children + i; - if (strcasecmp ("URL", child->key) == 0) + if (strcasecmp ("Sock", child->key) == 0 + || strcasecmp ("URL", child->key) == 0) { status = cj_config_add_url (child); if (status == 0) @@ -779,11 +817,10 @@ static void cj_submit (cj_t *db, cj_key_t *key, value_t *value) /* {{{ */ if (key->instance == NULL) { - if ((db->depth == 0) || (strcmp ("", db->state[db->depth-1].name) == 0)) - sstrncpy (vl.type_instance, db->state[db->depth].name, sizeof (vl.type_instance)); - else - ssnprintf (vl.type_instance, sizeof (vl.type_instance), "%s-%s", - db->state[db->depth-1].name, db->state[db->depth].name); + int i, len = 0; + for (i = 0; i < db->depth; i++) + len += ssnprintf(vl.type_instance+len, sizeof(vl.type_instance)-len, + i ? "-%s" : "%s", db->state[i+1].name); } else sstrncpy (vl.type_instance, key->instance, sizeof (vl.type_instance)); @@ -793,14 +830,81 @@ static void cj_submit (cj_t *db, cj_key_t *key, value_t *value) /* {{{ */ sstrncpy (vl.plugin_instance, db->instance, sizeof (vl.plugin_instance)); sstrncpy (vl.type, key->type, sizeof (vl.type)); + if (db->interval > 0) + vl.interval = db->interval; + plugin_dispatch_values (&vl); } /* }}} int cj_submit */ -static int cj_curl_perform (cj_t *db, CURL *curl) /* {{{ */ +static int cj_sock_perform (cj_t *db) /* {{{ */ +{ + char errbuf[1024]; + struct sockaddr_un sa_unix = {}; + sa_unix.sun_family = AF_UNIX; + sstrncpy (sa_unix.sun_path, db->sock, sizeof (sa_unix.sun_path)); + + int fd = socket (AF_UNIX, SOCK_STREAM, 0); + if (fd < 0) + return (-1); + if (connect (fd, (struct sockaddr *)&sa_unix, sizeof(sa_unix)) < 0) + { + ERROR ("curl_json plugin: connect(%s) failed: %s", + (db->sock != NULL) ? db->sock : "", + sstrerror(errno, errbuf, sizeof (errbuf))); + close (fd); + return (-1); + } + + ssize_t red; + do { + unsigned char buffer[4096]; + red = read (fd, buffer, sizeof(buffer)); + if (red < 0) { + ERROR ("curl_json plugin: read(%s) failed: %s", + (db->sock != NULL) ? db->sock : "", + sstrerror(errno, errbuf, sizeof (errbuf))); + close (fd); + return (-1); + } + if (!cj_curl_callback (buffer, red, 1, db)) + break; + } while (red > 0); + close (fd); + return (0); +} /* }}} int cj_sock_perform */ + + +static int cj_curl_perform(cj_t *db) /* {{{ */ { int status; long rc; char *url; + url = db->url; + + status = curl_easy_perform (db->curl); + if (status != CURLE_OK) + { + ERROR ("curl_json plugin: curl_easy_perform failed with status %i: %s (%s)", + status, db->curl_errbuf, url); + return (-1); + } + + curl_easy_getinfo(db->curl, CURLINFO_EFFECTIVE_URL, &url); + curl_easy_getinfo(db->curl, CURLINFO_RESPONSE_CODE, &rc); + + /* The response code is zero if a non-HTTP transport was used. */ + if ((rc != 0) && (rc != 200)) + { + ERROR ("curl_json plugin: curl_easy_perform failed with " + "response code %ld (%s)", rc, url); + return (-1); + } + return (0); +} /* }}} int cj_curl_perform */ + +static int cj_perform (cj_t *db) /* {{{ */ +{ + int status; yajl_handle yprev = db->yajl; db->yajl = yajl_alloc (&ycallbacks, @@ -817,50 +921,40 @@ static int cj_curl_perform (cj_t *db, CURL *curl) /* {{{ */ return (-1); } - status = curl_easy_perform (curl); - if (status != CURLE_OK) - { - ERROR ("curl_json plugin: curl_easy_perform failed with status %i: %s (%s)", - status, db->curl_errbuf, url); - yajl_free (db->yajl); - db->yajl = yprev; - return (-1); - } - - curl_easy_getinfo(curl, CURLINFO_EFFECTIVE_URL, &url); - curl_easy_getinfo(curl, CURLINFO_RESPONSE_CODE, &rc); - - /* The response code is zero if a non-HTTP transport was used. */ - if ((rc != 0) && (rc != 200)) + if (db->url) + status = cj_curl_perform (db); + else + status = cj_sock_perform (db); + if (status < 0) { - ERROR ("curl_json plugin: curl_easy_perform failed with response code %ld (%s)", - rc, url); yajl_free (db->yajl); db->yajl = yprev; return (-1); } #if HAVE_YAJL_V2 - status = yajl_complete_parse(db->yajl); + status = yajl_complete_parse(db->yajl); #else - status = yajl_parse_complete(db->yajl); + status = yajl_parse_complete(db->yajl); #endif if (status != yajl_status_ok) { - ERROR ("curl_json plugin: %s failed with status %i.", -#if HAVE_YAJL_V2 - "yajl_complete_parse", -#else - "yajl_parse_complete", -#endif - status); + unsigned char *errmsg; + + errmsg = yajl_get_error (db->yajl, /* verbose = */ 0, + /* jsonText = */ NULL, /* jsonTextLen = */ 0); + ERROR ("curl_json plugin: yajl_parse_complete failed: %s", + (char *) errmsg); + yajl_free_error (db->yajl, errmsg); + yajl_free (db->yajl); + db->yajl = yprev; + return (-1); } yajl_free (db->yajl); db->yajl = yprev; - return (0); -} /* }}} int cj_curl_perform */ +} /* }}} int cj_perform */ static int cj_read (user_data_t *ud) /* {{{ */ { @@ -879,12 +973,21 @@ static int cj_read (user_data_t *ud) /* {{{ */ db->state[db->depth].tree = db->tree; db->key = NULL; - return cj_curl_perform (db, db->curl); + return cj_perform (db); } /* }}} int cj_read */ +static int cj_init (void) /* {{{ */ +{ + /* Call this while collectd is still single-threaded to avoid + * initialization issues in libgcrypt. */ + curl_global_init (CURL_GLOBAL_SSL); + return (0); +} /* }}} int cj_init */ + void module_register (void) { plugin_register_complex_config ("curl_json", cj_config); + plugin_register_init ("curl_json", cj_init); } /* void module_register */ /* vim: set sw=2 sts=2 et fdm=marker : */