Changeset b1dc403 for protocols/jabber/sasl.c

Timestamp:

2015-05-04T21:58:50Z (9 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Children:

5726a0d

Parents:

531eabd (diff), 5ca1416 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Catch up with master.

File:

• protocols/jabber/sasl.c (modified) (6 diffs)

protocols/jabber/sasl.c

@@ -42 +42 @@
         "https://www.facebook.com/dialog/oauth",
         "https://graph.facebook.com/oauth/access_token",
-        "http://www.bitlbee.org/main.php/Facebook/oauth2.html",
+        "https://www.bitlbee.org/main.php/Facebook/oauth2.html",
         "offline_access,xmpp_login",
         "126828914005625",
@@ -55 +55 @@
         char *s;
         int sup_plain = 0, sup_digest = 0, sup_gtalk = 0, sup_fb = 0;
-        int want_oauth = FALSE;
+        int want_oauth = FALSE, want_hipchat = FALSE;
         GString *mechs;
 
@@ -75 +75 @@
 
         want_oauth = set_getbool(&ic->acc->set, "oauth");
+        want_hipchat = (jd->flags & JFLAG_HIPCHAT);
 
         mechs = g_string_new("");
@@ -111 +112 @@
 
         reply = xt_new_node("auth", NULL, NULL);
-        xt_add_attr(reply, "xmlns", XMLNS_SASL);
+        if (!want_hipchat) {
+                xt_add_attr(reply, "xmlns", XMLNS_SASL);
+        } else {
+                xt_add_attr(reply, "xmlns", XMLNS_HIPCHAT);
+        }
 
         if (sup_gtalk && want_oauth) {
@@ -143 +148 @@
         } else if (sup_plain) {
                 int len;
-
-                xt_add_attr(reply, "mechanism", "PLAIN");
+                GString *gs;
+                char *username;
+
+                if (!want_hipchat) {
+                        xt_add_attr(reply, "mechanism", "PLAIN");
+                        username = jd->username;
+                } else {
+                        username = jd->me;
+                }
+
+                /* set an arbitrary initial size to avoid reallocations */
+                gs = g_string_sized_new(128);
 
                 /* With SASL PLAIN in XMPP, the text should be b64(\0user\0pass) */
-                len = strlen(jd->username) + strlen(ic->acc->pass) + 2;
-                s = g_malloc(len + 1);
-                s[0] = 0;
-                strcpy(s + 1, jd->username);
-                strcpy(s + 2 + strlen(jd->username), ic->acc->pass);
+                g_string_append_c(gs, '\0');
+                g_string_append(gs, username);
+                g_string_append_c(gs, '\0');
+                g_string_append(gs, ic->acc->pass);
+                if (want_hipchat) {
+                        /* Hipchat's variation adds \0resource at the end */
+                        g_string_append_c(gs, '\0');
+                        g_string_append(gs, set_getstr(&ic->acc->set, "resource"));
+                }
+
+                len = gs->len;
+                s = g_string_free(gs, FALSE);
+
                 reply->text = base64_encode((unsigned char *) s, len);
                 reply->text_len = strlen(reply->text);
@@ -397 +420 @@
                 imcb_log(ic, "Authentication finished");
                 jd->flags |= JFLAG_AUTHENTICATED | JFLAG_STREAM_RESTART;
+
+                if (jd->flags & JFLAG_HIPCHAT) {
+                        return hipchat_handle_success(ic, node);
+                }
         } else if (strcmp(node->name, "failure") == 0) {
                 imcb_error(ic, "Authentication failure");