Matej Habrnal e06906
From ad37a7f1b14ec42022df14913dcfa704f829da03 Mon Sep 17 00:00:00 2001
Matej Habrnal e06906
From: Matej Habrnal <mhabrnal@redhat.com>
Matej Habrnal e06906
Date: Wed, 26 Aug 2015 13:41:27 +0200
Matej Habrnal e06906
Subject: [PATCH] bodhi: add parsing of error responses
Matej Habrnal e06906
Matej Habrnal e06906
Resolves: rhbz#1256493
Matej Habrnal e06906
Matej Habrnal e06906
Signed-off-by: Matej Habrnal <mhabrnal@redhat.com>
Matej Habrnal e06906
---
Matej Habrnal e06906
 src/plugins/bodhi.c | 70 ++++++++++++++++++++++++++++++++++++++++++++++++++++-
Matej Habrnal e06906
 1 file changed, 69 insertions(+), 1 deletion(-)
Matej Habrnal e06906
Matej Habrnal e06906
diff --git a/src/plugins/bodhi.c b/src/plugins/bodhi.c
Matej Habrnal e06906
index 9149347..b348a26 100644
Matej Habrnal e06906
--- a/src/plugins/bodhi.c
Matej Habrnal e06906
+++ b/src/plugins/bodhi.c
Matej Habrnal e06906
@@ -217,6 +217,58 @@ static void print_bodhi(struct bodhi *b)
Matej Habrnal e06906
 }
Matej Habrnal e06906
 #endif
Matej Habrnal e06906
 
Matej Habrnal e06906
+/* bodhi returns following json structure in case of error
Matej Habrnal e06906
+{
Matej Habrnal e06906
+   "status": "error",
Matej Habrnal e06906
+   "errors":
Matej Habrnal e06906
+              [
Matej Habrnal e06906
+                {
Matej Habrnal e06906
+                   "location": "querystring",
Matej Habrnal e06906
+                   "name": "releases",
Matej Habrnal e06906
+                   "description": "Invalid releases specified: Rawhide"
Matej Habrnal e06906
+                }
Matej Habrnal e06906
+              ]
Matej Habrnal e06906
+}
Matej Habrnal e06906
+*/
Matej Habrnal e06906
+static void bodhi_print_errors_from_json(json_object *json)
Matej Habrnal e06906
+{
Matej Habrnal e06906
+
Matej Habrnal e06906
+    json_object *errors_array = NULL;
Matej Habrnal e06906
+    bodhi_read_value(json, "errors", &errors_array, BODHI_READ_JSON_OBJ);
Matej Habrnal e06906
+    if (!errors_array)
Matej Habrnal e06906
+    {
Matej Habrnal e06906
+        error_msg("Error: unable to read 'errors' array from json");
Matej Habrnal e06906
+        return;
Matej Habrnal e06906
+    }
Matej Habrnal e06906
+
Matej Habrnal e06906
+    int errors_len = json_object_array_length(errors_array);
Matej Habrnal e06906
+    for (int i = 0; i < errors_len; ++i)
Matej Habrnal e06906
+    {
Matej Habrnal e06906
+        json_object *error = json_object_array_get_idx(errors_array, i);
Matej Habrnal e06906
+        if (!error)
Matej Habrnal e06906
+        {
Matej Habrnal e06906
+            error_msg("Error: unable to get 'error[%d]'", i);
Matej Habrnal e06906
+            json_object_put(errors_array);
Matej Habrnal e06906
+            return;
Matej Habrnal e06906
+        }
Matej Habrnal e06906
+
Matej Habrnal e06906
+        char *desc_item = NULL;
Matej Habrnal e06906
+        bodhi_read_value(error, "description", &desc_item, BODHI_READ_STR);
Matej Habrnal e06906
+        if (!desc_item)
Matej Habrnal e06906
+        {
Matej Habrnal e06906
+            error_msg("Error: unable to get 'description' from 'error[%d]'", i);
Matej Habrnal e06906
+            continue;
Matej Habrnal e06906
+        }
Matej Habrnal e06906
+
Matej Habrnal e06906
+        error_msg("Error: %s", desc_item);
Matej Habrnal e06906
+        json_object_put(error);
Matej Habrnal e06906
+        free(desc_item);
Matej Habrnal e06906
+    }
Matej Habrnal e06906
+
Matej Habrnal e06906
+    json_object_put(errors_array);
Matej Habrnal e06906
+    return;
Matej Habrnal e06906
+}
Matej Habrnal e06906
+
Matej Habrnal e06906
 static GHashTable *bodhi_parse_json(json_object *json, const char *release)
Matej Habrnal e06906
 {
Matej Habrnal e06906
 
Matej Habrnal e06906
@@ -326,7 +378,7 @@ static GHashTable *bodhi_query_list(const char *query, const char *release)
Matej Habrnal e06906
     get(post_state, bodhi_url_bugs, "application/x-www-form-urlencoded",
Matej Habrnal e06906
                      headers);
Matej Habrnal e06906
 
Matej Habrnal e06906
-    if (post_state->http_resp_code != 200)
Matej Habrnal e06906
+    if (post_state->http_resp_code != 200 && post_state->http_resp_code != 400)
Matej Habrnal e06906
     {
Matej Habrnal e06906
         char *errmsg = post_state->curl_error_msg;
Matej Habrnal e06906
         if (errmsg && errmsg[0])
Matej Habrnal e06906
@@ -340,6 +392,22 @@ static GHashTable *bodhi_query_list(const char *query, const char *release)
Matej Habrnal e06906
     if (is_error(json))
Matej Habrnal e06906
         error_msg_and_die("fatal: unable parse response from bodhi server");
Matej Habrnal e06906
 
Matej Habrnal e06906
+    /* we must check the http_resp_code because only error responses contain
Matej Habrnal e06906
+     * 'status' item. 'bodhi_read_value' function prints an error message in
Matej Habrnal e06906
+     * the case it did not found the item */
Matej Habrnal e06906
+    if (post_state->http_resp_code != 200)
Matej Habrnal e06906
+    {
Matej Habrnal e06906
+        char *status_item = NULL;
Matej Habrnal e06906
+        bodhi_read_value(json, "status", &status_item, BODHI_READ_STR);
Matej Habrnal e06906
+        if (status_item != NULL && strcmp(status_item, "error") == 0)
Matej Habrnal e06906
+        {
Matej Habrnal e06906
+            free(status_item);
Matej Habrnal e06906
+            bodhi_print_errors_from_json(json);
Matej Habrnal e06906
+            json_object_put(json);
Matej Habrnal e06906
+            xfunc_die(); // error_msg are printed in bodhi_print_errors_from_json
Matej Habrnal e06906
+        }
Matej Habrnal e06906
+    }
Matej Habrnal e06906
+
Matej Habrnal e06906
     GHashTable *bodhi_table = bodhi_parse_json(json, release);
Matej Habrnal e06906
     json_object_put(json);
Matej Habrnal e06906
     free_post_state(post_state);
Matej Habrnal e06906
-- 
Matej Habrnal e06906
2.5.0
Matej Habrnal e06906