Code

data: Double quote formatted date-times.
[sysdb.git] / src / core / store.c
index d195b8f0e3d029f4bf853910ba4ce05358e85d58..2522233112a8d125024d9853710ff90565f768d4 100644 (file)
@@ -57,6 +57,11 @@ static sdb_type_t sdb_attribute_type;
 
 struct sdb_store_base {
        sdb_object_t super;
+
+       /* object type */
+       int type;
+
+       /* common meta information */
        sdb_time_t last_update;
        sdb_store_base_t *parent;
 };
@@ -66,7 +71,7 @@ struct sdb_store_base {
 typedef struct {
        sdb_store_base_t super;
 
-       char *value;
+       sdb_data_t value;
 } sdb_attribute_t;
 #define SDB_ATTR(obj) ((sdb_attribute_t *)(obj))
 #define SDB_CONST_ATTR(obj) ((const sdb_attribute_t *)(obj))
@@ -74,9 +79,7 @@ typedef struct {
 typedef struct {
        sdb_store_base_t super;
 
-       int type;
        sdb_llist_t *children;
-
        sdb_llist_t *attributes;
 } sdb_store_obj_t;
 #define SDB_STORE_OBJ(obj) ((sdb_store_obj_t *)(obj))
@@ -100,8 +103,10 @@ static int
 store_base_init(sdb_object_t *obj, va_list ap)
 {
        sdb_store_base_t *sobj = STORE_BASE(obj);
-       sobj->last_update = va_arg(ap, sdb_time_t);
 
+       sobj->type = va_arg(ap, int);
+
+       sobj->last_update = va_arg(ap, sdb_time_t);
        sobj->parent = NULL;
        return 0;
 } /* store_base_init */
@@ -121,12 +126,11 @@ sdb_store_obj_init(sdb_object_t *obj, va_list ap)
        sdb_store_obj_t *sobj = SDB_STORE_OBJ(obj);
        int ret;
 
+       /* this will consume the first argument (type) of ap */
        ret = store_base_init(obj, ap);
        if (ret)
                return ret;
 
-       sobj->type = va_arg(ap, int);
-
        sobj->children = sdb_llist_create();
        if (! sobj->children)
                return -1;
@@ -154,19 +158,19 @@ sdb_store_obj_destroy(sdb_object_t *obj)
 static int
 sdb_attr_init(sdb_object_t *obj, va_list ap)
 {
-       const char *value;
+       const sdb_data_t *value;
        int ret;
 
+       /* this will consume the first two arguments
+        * (type and last_update) of ap */
        ret = store_base_init(obj, ap);
        if (ret)
                return ret;
-       value = va_arg(ap, const char *);
+       value = va_arg(ap, const sdb_data_t *);
 
-       if (value) {
-               SDB_ATTR(obj)->value = strdup(value);
-               if (! SDB_ATTR(obj)->value)
+       if (value)
+               if (sdb_data_copy(&SDB_ATTR(obj)->value, value))
                        return -1;
-       }
        return 0;
 } /* sdb_attr_init */
 
@@ -176,9 +180,7 @@ sdb_attr_destroy(sdb_object_t *obj)
        assert(obj);
 
        store_base_destroy(obj);
-
-       if (SDB_ATTR(obj)->value)
-               free(SDB_ATTR(obj)->value);
+       sdb_data_free_datum(&SDB_ATTR(obj)->value);
 } /* sdb_attr_destroy */
 
 static sdb_type_t sdb_store_obj_type = {
@@ -215,14 +217,14 @@ sdb_store_lookup_in_list(sdb_llist_t *l, int type, const char *name)
                sdb_store_obj_t *sobj = SDB_STORE_OBJ(sdb_llist_iter_get_next(iter));
                assert(sobj);
 
-               if ((sobj->type == type)
+               if ((STORE_BASE(sobj)->type == type)
                                && (! strcasecmp(SDB_OBJ(sobj)->name, name))) {
                        sdb_llist_iter_destroy(iter);
                        return sobj;
                }
 
                /* don't lookups non-host types from hierarchical hosts */
-               if ((type != SDB_HOST) && (sobj->type == SDB_HOST))
+               if ((type != SDB_HOST) && (STORE_BASE(sobj)->type == SDB_HOST))
                        continue;
 
                sobj = sdb_store_lookup_in_list(sobj->children, type, name);
@@ -342,10 +344,10 @@ store_obj(int parent_type, const char *parent_name,
                if (type == SDB_ATTRIBUTE)
                        /* the value will be updated by the caller */
                        new = STORE_BASE(sdb_object_create(name, sdb_attribute_type,
-                                               last_update, NULL));
+                                               type, last_update, NULL));
                else
                        new = STORE_BASE(sdb_object_create(name, sdb_store_obj_type,
-                                               last_update, type));
+                                               type, last_update));
 
                if (! new) {
                        char errbuf[1024];
@@ -371,7 +373,58 @@ store_obj(int parent_type, const char *parent_name,
        free(parent_cname);
        free(cname);
        return status;
-} /* sdb_store_obj */
+} /* store_obj */
+
+/*
+ * store_obj_tojson serializes attribute / service objects to JSON.
+ *
+ * The function never returns an error. Rather, an error message will be part
+ * of the serialized data.
+ */
+static void
+store_obj_tojson(sdb_llist_t *list, int type, sdb_strbuf_t *buf)
+{
+       sdb_llist_iter_t *iter;
+       char time_str[64];
+
+       assert((type == SDB_ATTRIBUTE) || (type == SDB_SERVICE));
+
+       sdb_strbuf_append(buf, "[");
+
+       iter = sdb_llist_get_iter(list);
+       if (! iter) {
+               char errbuf[1024];
+               sdb_log(SDB_LOG_ERR, "store: Failed to retrieve %ss: %s\n",
+                               TYPE_TO_NAME(type),
+                               sdb_strerror(errno, errbuf, sizeof(errbuf)));
+               sdb_strbuf_append(buf, "{\"error\": \"failed to retrieve %ss: %s\"}",
+                               TYPE_TO_NAME(type), errbuf);
+       }
+
+       /* has_next returns false if the iterator is NULL */
+       while (sdb_llist_iter_has_next(iter)) {
+               sdb_store_base_t *sobj = STORE_BASE(sdb_llist_iter_get_next(iter));
+               assert(sobj);
+
+               if (! sdb_strftime(time_str, sizeof(time_str),
+                                       "%F %T %z", sobj->last_update))
+                       snprintf(time_str, sizeof(time_str), "<error>");
+               time_str[sizeof(time_str) - 1] = '\0';
+
+               sdb_strbuf_append(buf, "{\"name\": \"%s\", ", SDB_OBJ(sobj)->name);
+               if (type == SDB_ATTRIBUTE)
+                       /* XXX: this needs to be type-dependent */
+                       sdb_strbuf_append(buf, "\"value\": \"%s\", ",
+                                       SDB_ATTR(sobj)->value.data.string);
+               sdb_strbuf_append(buf, "\"last_update\": \"%s\"}", time_str);
+
+               if (sdb_llist_iter_has_next(iter))
+                       sdb_strbuf_append(buf, ",");
+       }
+
+       sdb_llist_iter_destroy(iter);
+       sdb_strbuf_append(buf, "]");
+} /* store_obj_tojson */
 
 /*
  * public API
@@ -405,8 +458,25 @@ sdb_store_has_host(const char *name)
        return host != NULL;
 } /* sdb_store_has_host */
 
+sdb_store_base_t *
+sdb_store_get_host(const char *name)
+{
+       sdb_store_obj_t *host;
+
+       if (! name)
+               return NULL;
+
+       host = sdb_store_lookup(SDB_HOST, name);
+       if (! host)
+               return NULL;
+
+       sdb_object_ref(SDB_OBJ(host));
+       return STORE_BASE(host);
+} /* sdb_store_get_host */
+
 int
-sdb_store_attribute(const char *hostname, const char *key, const char *value,
+sdb_store_attribute(const char *hostname,
+               const char *key, const sdb_data_t *value,
                sdb_time_t last_update)
 {
        int status;
@@ -423,8 +493,7 @@ sdb_store_attribute(const char *hostname, const char *key, const char *value,
 
        if (status >= 0) {
                assert(updated_attr);
-               SDB_ATTR(updated_attr)->value = strdup(value);
-               if (! SDB_ATTR(updated_attr)->value) {
+               if (sdb_data_copy(&SDB_ATTR(updated_attr)->value, value)) {
                        sdb_object_deref(SDB_OBJ(updated_attr));
                        status = -1;
                }
@@ -451,9 +520,43 @@ sdb_store_service(const char *hostname, const char *name,
        return status;
 } /* sdb_store_service */
 
+int
+sdb_store_host_tojson(sdb_store_base_t *h, sdb_strbuf_t *buf, int flags)
+{
+       sdb_store_obj_t *host;
+       char time_str[64];
+
+       if ((! h) || (h->type != SDB_HOST) || (! buf))
+               return -1;
+
+       host = SDB_STORE_OBJ(h);
+
+       if (! sdb_strftime(time_str, sizeof(time_str),
+                               "%F %T %z", host->_last_update))
+               snprintf(time_str, sizeof(time_str), "<error>");
+       time_str[sizeof(time_str) - 1] = '\0';
+
+       sdb_strbuf_append(buf, "{\"name\": \"%s\", "
+                       "\"last_update\": \"%s\"",
+                       SDB_OBJ(host)->name, time_str);
+
+       if (! (flags & SDB_SKIP_ATTRIBUTES)) {
+               sdb_strbuf_append(buf, ", \"attributes\": ");
+               store_obj_tojson(host->attributes, SDB_ATTRIBUTE, buf);
+       }
+
+       if (! (flags & SDB_SKIP_SERVICES)) {
+               sdb_strbuf_append(buf, ", \"services\": ");
+               store_obj_tojson(host->children, SDB_SERVICE, buf);
+       }
+
+       sdb_strbuf_append(buf, "}");
+       return 0;
+} /* sdb_store_host_tojson */
+
 /* TODO: actually support hierarchical data */
 int
-sdb_store_tojson(sdb_strbuf_t *buf)
+sdb_store_tojson(sdb_strbuf_t *buf, int flags)
 {
        sdb_llist_iter_t *host_iter;
 
@@ -471,76 +574,14 @@ sdb_store_tojson(sdb_strbuf_t *buf)
        sdb_strbuf_append(buf, "{\"hosts\":[");
 
        while (sdb_llist_iter_has_next(host_iter)) {
-               sdb_store_obj_t *host = SDB_STORE_OBJ(sdb_llist_iter_get_next(host_iter));
-               sdb_llist_iter_t *svc_iter;
-               sdb_llist_iter_t *attr_iter;
-
-               char time_str[64];
-
+               sdb_store_base_t *host = STORE_BASE(sdb_llist_iter_get_next(host_iter));
                assert(host);
 
-               if (! sdb_strftime(time_str, sizeof(time_str),
-                                       "%F %T %z", host->_last_update))
-                       snprintf(time_str, sizeof(time_str), "<error>");
-               time_str[sizeof(time_str) - 1] = '\0';
-
-               sdb_strbuf_append(buf, "{\"name\": \"%s\", "
-                               "\"last_update\": \"%s\", "
-                               "\"attributes\": [",
-                               SDB_OBJ(host)->name, time_str);
-
-               attr_iter = sdb_llist_get_iter(host->attributes);
-               if (! attr_iter) {
-                       char errbuf[1024];
-                       sdb_log(SDB_LOG_ERR, "store: Failed to retrieve attributes: %s\n",
-                                       sdb_strerror(errno, errbuf, sizeof(errbuf)));
-                       sdb_strbuf_append(buf, "{\"error\": \"failed to retrieve "
-                                       "attributes: %s\"}", errbuf);
-               }
-
-               /* has_next returns false if the iterator is NULL */
-               while (sdb_llist_iter_has_next(attr_iter)) {
-                       sdb_attribute_t *attr = SDB_ATTR(sdb_llist_iter_get_next(attr_iter));
-                       assert(attr);
-
-                       if (! sdb_strftime(time_str, sizeof(time_str),
-                                               "%F %T %z", attr->_last_update))
-                               snprintf(time_str, sizeof(time_str), "<error>");
-                       time_str[sizeof(time_str) - 1] = '\0';
-
-                       sdb_strbuf_append(buf, "{\"name\": \"%s\", "
-                                       "\"value\": \"%s\", \"last_update\": \"%s\"},",
-                                       SDB_OBJ(attr)->name, attr->value, time_str);
-               }
-
-               sdb_llist_iter_destroy(attr_iter);
-               sdb_strbuf_append(buf, "], \"services\": [");
-
-               svc_iter = sdb_llist_get_iter(host->children);
-               if (! svc_iter) {
-                       char errbuf[1024];
-                       sdb_log(SDB_LOG_ERR, "store: Failed to retrieve services: %s\n",
-                                       sdb_strerror(errno, errbuf, sizeof(errbuf)));
-                       sdb_strbuf_append(buf, "{\"error\": \"failed to retrieve "
-                                       "services: %s\"}", errbuf);
-               }
-
-               while (sdb_llist_iter_has_next(svc_iter)) {
-                       sdb_store_obj_t *svc = SDB_STORE_OBJ(sdb_llist_iter_get_next(svc_iter));
-                       assert(svc);
-
-                       if (! sdb_strftime(time_str, sizeof(time_str),
-                                               "%F %T %z", svc->_last_update))
-                               snprintf(time_str, sizeof(time_str), "<error>");
-                       time_str[sizeof(time_str) - 1] = '\0';
-
-                       sdb_strbuf_append(buf, "{\"name\": \"%s\", "
-                                       "\"last_update\": \"%s\"},",
-                                       SDB_OBJ(svc)->name, time_str);
-               }
+               if (sdb_store_host_tojson(host, buf, flags))
+                       return -1;
 
-               sdb_llist_iter_destroy(svc_iter);
-               sdb_strbuf_append(buf, "]},");
+               if (sdb_llist_iter_has_next(host_iter))
+                       sdb_strbuf_append(buf, ",");
        }
 
        sdb_strbuf_append(buf, "]}");