Changes in protocols/jabber/jabber.c [3a620ff:73dd021]

File:

• protocols/jabber/jabber.c (modified) (14 diffs)

protocols/jabber/jabber.c

@@ -114 +114 @@
         s->flags |= ACC_SET_OFFLINE_ONLY | SET_NULL_OK;
 
-        s = set_add(&acc->set, "carbons", "true", set_eval_bool, acc);
-        s->flags |= ACC_SET_OFFLINE_ONLY;
-
         acc->flags |= ACC_FLAG_AWAY_MESSAGE | ACC_FLAG_STATUS_MESSAGE |
                       ACC_FLAG_HANDLE_DOMAINS;
@@ -151 +148 @@
         }
 
-        if (strstr(jd->server, ".facebook.com")) {
-                imcb_error(ic, "Facebook's XMPP service is gone. Try this instead: https://wiki.bitlbee.org/HowtoFacebookMQTT");
-                imc_logout(ic, FALSE);
-                return;
-        }
-
         if ((s = strchr(jd->server, '/'))) {
                 *s = 0;
@@ -176 +167 @@
                 jd->fd = jd->r_inpa = jd->w_inpa = -1;
 
-                /* There are no other options atm, so assume google for everything 
-                   Facebook and MSN XMPP used to be here. RIP. */
-                jd->oauth2_service = &oauth2_service_google;
+                if (strstr(jd->server, ".facebook.com")) {
+                        jd->oauth2_service = &oauth2_service_facebook;
+                } else {
+                        jd->oauth2_service = &oauth2_service_google;
+                }
 
                 oauth_params_parse(&p_in, ic->acc->pass);
@@ -204 +197 @@
         } else {
                 jabber_connect(ic);
-        }
-}
-
-static void jabber_xmlconsole_enable(struct im_connection *ic)
-{
-        struct jabber_data *jd = ic->proto_data;
-        const char *handle = JABBER_XMLCONSOLE_HANDLE;
-        bee_user_t *bu;
-        
-        jd->flags |= JFLAG_XMLCONSOLE;
-
-        if (!(bu = bee_user_by_handle(ic->bee, ic, handle))) {
-                bu = bee_user_new(ic->bee, ic, handle, 0);
-                bu->flags |= BEE_USER_NOOTR;
         }
 }
@@ -287 +266 @@
 
         if (set_getbool(&acc->set, "xmlconsole")) {
-                jabber_xmlconsole_enable(ic);
+                jd->flags |= JFLAG_XMLCONSOLE;
+                /* Shouldn't really do this at this stage already, maybe. But
+                   I think this shouldn't break anything. */
+                imcb_add_buddy(ic, JABBER_XMLCONSOLE_HANDLE, NULL);
         }
 
@@ -352 +334 @@
         }
         if (jd->fd >= 0) {
-                proxy_disconnect(jd->fd);
+                closesocket(jd->fd);
         }
 
@@ -363 +345 @@
         }
 
-        if (jd->buddies) {
-                jabber_buddy_remove_all(ic);
-        }
+        jabber_buddy_remove_all(ic);
 
         xt_free(jd->xt);
@@ -375 +355 @@
         g_free(jd->internal_jid);
         g_free(jd->gmail_tid);
-        g_free(jd->muc_host);
         g_free(jd->username);
         g_free(jd->me);
@@ -434 +413 @@
         }
 
-        /* XEP-0364 suggests we add message processing hints (XEP-0334) to OTR messages,
-           mostly to avoid carbons (XEP-0280) and server-side message archiving.
-           OTR messages are roughly like this: /^\?OTR(.*\?| Error:|:)/
-           But I'm going to simplify it to messages starting with "?OTR". */
-        if (g_str_has_prefix(message, "?OTR")) {
-                int i;
-                char *hints[] = {
-                        "no-copy", XMLNS_HINTS,
-                        "no-permanent-store", XMLNS_HINTS,
-                        "private", XMLNS_CARBONS,
-                        NULL
-                };
-                        
-                for (i = 0; hints[i]; i += 2) {
-                        struct xt_node *hint;
-                        hint = xt_new_node(hints[i], NULL, NULL);
-                        xt_add_attr(hint, "xmlns", hints[i + 1]);
-                        xt_add_child(node, hint);
-                }
-        }
-
         st = jabber_write_packet(ic, node);
         xt_free_node(node);
@@ -514 +472 @@
 static void jabber_add_buddy(struct im_connection *ic, char *who, char *group)
 {
+        struct jabber_data *jd = ic->proto_data;
+
         if (g_strcasecmp(who, JABBER_XMLCONSOLE_HANDLE) == 0) {
-                jabber_xmlconsole_enable(ic);
+                jd->flags |= JFLAG_XMLCONSOLE;
+                imcb_add_buddy(ic, JABBER_XMLCONSOLE_HANDLE, NULL);
                 return;
         }
@@ -559 +520 @@
         }
 
-        if (jd->flags & JFLAG_HIPCHAT && jd->muc_host && !g_str_has_suffix(room, jd->muc_host)) {
-                char *guessed_name = hipchat_guess_channel_name(ic, room);
-                if (guessed_name) {
-                        set_setstr(sets, "room", guessed_name);
-                        g_free(guessed_name);
-
-                        /* call this same function again with the fixed name */
-                        return jabber_chat_join_(ic, set_getstr(sets, "room"), nick, password, sets);
-                }
-        }
-
         if (strchr(room, '@') == NULL) {
                 imcb_error(ic, "%s is not a valid Jabber room name. Maybe you mean %s@conference.%s?",
@@ -576 +526 @@
                 imcb_error(ic, "Already present in chat `%s'", room);
         } else {
-                /* jabber_chat_join without the underscore is the conference.c one */
                 return jabber_chat_join(ic, room, final_nick, set_getstr(sets, "password"));
         }
@@ -641 +590 @@
 {
         struct jabber_data *jd = ic->proto_data;
-        struct jabber_buddy *bud, *bare;
+        struct jabber_buddy *bud;
 
         /* Enable typing notification related code from now. */
         jd->flags |= JFLAG_WANT_TYPING;
 
-        if ((bud = jabber_buddy_by_jid(ic, who, 0)) == NULL ||
-            (bare = jabber_buddy_by_jid(ic, who, GET_BUDDY_BARE)) == NULL) {
+        if ((bud = jabber_buddy_by_jid(ic, who, 0)) == NULL) {
                 /* Sending typing notifications to unknown buddies is
                    unsupported for now. Shouldn't be a problem, I think. */
@@ -653 +601 @@
         }
 
-
-        if (bud->flags & JBFLAG_DOES_XEP85 || bare->flags & JBFLAG_DOES_XEP85) {
+        if (bud->flags & JBFLAG_DOES_XEP85) {
                 /* We're only allowed to send this stuff if we know the other
-                   side supports it. If the bare JID has the flag, all other
-                   resources get it, too (That is the case in gtalk) */
+                   side supports it. */
 
                 struct xt_node *node;