Changes in lib/oauth2.c [1a81c83:5535a47]

File:

• lib/oauth2.c (modified) (4 diffs)

lib/oauth2.c

@@ -42 +42 @@
 #include "oauth2.h"
 #include "oauth.h"
-#include "parson.h"
+#include "json.h"
+#include "json_util.h"
 #include "url.h"
-
-#define JSON_O_FOREACH(o, k, v) \
-    const char *k; const JSON_Value *v; int __i; \
-    for (__i = 0; json_object_get_tuple(o, __i, &k, &v); __i++)
 
 char *oauth2_url(const struct oauth2_service *sp)
@@ -116 +113 @@
 }
 
-static char* oauth2_parse_error(const JSON_Value *e)
+static char* oauth2_parse_error(json_value *e)
 {
         /* This does a reasonable job with some of the flavours of error
            responses I've seen. Because apparently it's not standardised. */
 
-        if (json_type(e) == JSONObject) {
+        if (e->type == json_object) {
                 /* Facebook style */
-                const char *msg = json_object_get_string(json_object(e), "message");
-                const char *type = json_object_get_string(json_object(e), "type");
-                JSON_Value *code_o = json_object_get_value(json_object(e), "code");
-                int code = json_value_get_integer(code_o);
+                const char *msg = json_o_str(e, "message");
+                const char *type = json_o_str(e, "type");
+                json_value *code_o = json_o_get(e, "code");
+                int code = 0;
+
+                if (code_o && code_o->type == json_integer) {
+                        code = code_o->u.integer;
+                }
+
                 return g_strdup_printf("Error %d: %s", code, msg ? msg : type ? type : "Unknown error");
-        } else if (json_type(e) == JSONString) {
-                return g_strdup(json_string(e));
+        } else if (e->type == json_string) {
+                return g_strdup(e->u.string.ptr);
         }
         return NULL;
@@ -138 +140 @@
         struct oauth2_access_token_data *cb_data = req->data;
         char *atoken = NULL, *rtoken = NULL, *error = NULL;
-        char *content_type;
+        char *content_type = NULL;
 
-        if (getenv("BITLBEE_DEBUG") && req->reply_body) {
+        if (req->status_code <= 0 && !req->reply_body) {
+                cb_data->func(cb_data->data, NULL, NULL, req->status_string);
+                g_free(cb_data);
+                return;
+        }
+
+        if (getenv("BITLBEE_DEBUG")) {
                 printf("%s\n", req->reply_body);
         }
@@ -148 +156 @@
         if (content_type && (strstr(content_type, "application/json") ||
                              strstr(content_type, "text/javascript"))) {
-                JSON_Value *js = json_parse_string(req->reply_body);
-                if (js && json_type(js) == JSONObject) {
-                        JSON_O_FOREACH(json_object(js), k, v){
+                json_value *js = json_parse(req->reply_body, req->body_size);
+                if (js && js->type == json_object) {
+                        JSON_O_FOREACH(js, k, v){
                                 if (strcmp(k, "error") == 0) {
                                         error = oauth2_parse_error(v);
                                 }
-                                if (json_type(v) != JSONString) {
+                                if (v->type != json_string) {
                                         continue;
                                 }
                                 if (strcmp(k, "access_token") == 0) {
-                                        atoken = g_strdup(json_string(v));
+                                        atoken = g_strdup(v->u.string.ptr);
                                 }
                                 if (strcmp(k, "refresh_token") == 0) {
-                                        rtoken = g_strdup(json_string(v));
+                                        rtoken = g_strdup(v->u.string.ptr);
                                 }
                         }