Changes in / [ec86b22:ca0981a]

Files:

• doc/user-guide/commands.xml (modified) (4 diffs)
• irc.c (modified) (1 diff)
• lib/Makefile (modified) (1 diff)
• lib/misc.c (modified) (1 diff)
• lib/oauth.c (deleted)
• lib/oauth.h (deleted)
• lib/url.c (modified) (1 diff)
• lib/url.h (modified) (1 diff)
• lib/xmltree.c (modified) (3 diffs)
• protocols/jabber/conference.c (modified) (1 diff)
• protocols/msn/sb.c (modified) (2 diffs)
• protocols/nogaim.c (modified) (5 diffs)
• protocols/oscar/oscar.c (modified) (4 diffs)
• protocols/twitter/twitter.c (modified) (7 diffs)
• protocols/twitter/twitter.h (modified) (1 diff)
• protocols/twitter/twitter_http.c (modified) (4 diffs)
• protocols/twitter/twitter_http.h (modified) (1 diff)
• protocols/twitter/twitter_lib.c (modified) (6 diffs)
• protocols/yahoo/yahoo.c (modified) (2 diffs)

doc/user-guide/commands.xml

@@ -79 +79 @@
                                                 To send tweets yourself, send them to the twitter_(yourusername) contact, or just write in the groupchat channel if you enabled that option.
                                         </para>
-
-                                        <para>
-                                                Since Twitter now requires OAuth authentication, you should not enter your Twitter password into BitlBee. Just type a bogus password. The first time you log in, BitlBee will start OAuth authentication. (See <emphasis>help set oauth</emphasis>.)
-                                        </para>
                                 </description>
                         </bitlbee-command>
@@ -663 +659 @@
         </bitlbee-setting>
 
-        <bitlbee-setting name="message_length" type="integer" scope="account">
-                <default>140</default>
-
-                <description>
-                        <para>
-                                Since Twitter rejects messages longer than 140 characters, BitlBee can count message length and emit a warning instead of waiting for Twitter to reject it.
-                        </para>
-
-                        <para>
-                                You can change this limit here but this won't disable length checks on Twitter's side. You can also set it to 0 to disable the check in case you believe BitlBee doesn't count the characters correctly.
-                        </para>
-                </description>
-
-        </bitlbee-setting>
-
         <bitlbee-setting name="mode" type="string" scope="account">
                 <possible-values>one, many, chat</possible-values>
@@ -722 +703 @@
         </bitlbee-setting>
 
-        <bitlbee-setting name="oauth" type="boolean" scope="account">
-                <default>true</default>
-
-                <description>
-                        <para>
-                                This enables OAuth authentication for Twitter accounts. From June 2010 this will be mandatory.
-                        </para>
-
-                        <para>
-                                With OAuth enabled, you shouldn't tell BitlBee your Twitter password. Just add your account with a bogus password and type <emphasis>account on</emphasis>. BitlBee will then give you a URL to authenticate with Twitter. If this succeeds, Twitter will return a PIN code which you can give back to BitlBee to finish the process.
-                        </para>
-
-                        <para>
-                                The resulting access token will be saved permanently, so you have to do this only once.
-                        </para>
-                </description>
-
-        </bitlbee-setting>
-
         <bitlbee-setting name="ops" type="string" scope="global">
                 <default>both</default>
@@ -872 +834 @@
                         <para>
                                 Can be set for Jabber- and OSCAR-connections. For Jabber, you might have to set this if the servername isn't equal to the part after the @ in the Jabber handle. For OSCAR this shouldn't be necessary anymore in recent BitlBee versions.
-                        </para>
-                </description>
-        </bitlbee-setting>
-
-        <bitlbee-setting name="show_offline" type="boolean" scope="global">
-                <default>false</default>
-
-                <description>
-                        <para>
-                                If enabled causes BitlBee to also show offline users in Channel. Online-users will get op, away-users voice and offline users none of both. This option takes effect as soon as you reconnect.
                         </para>
                 </description>

irc.c

@@ -202 +202 @@
         s = set_add( &irc->set, "root_nick", irc->mynick, set_eval_root_nick, irc );
         s = set_add( &irc->set, "save_on_quit", "true", set_eval_bool, irc );
-        s = set_add( &irc->set, "show_offline", "false", set_eval_bool, irc );
         s = set_add( &irc->set, "simulate_netsplit", "true", set_eval_bool, irc );
         s = set_add( &irc->set, "status", NULL,  set_eval_away_status, irc );

lib/Makefile

@@ -10 +10 @@
 
 # [SH] Program variables
-objects = arc.o base64.o $(EVENT_HANDLER) ftutil.o http_client.o ini.o md5.o misc.o oauth.o proxy.o sha1.o $(SSL_CLIENT) url.o xmltree.o
+objects = arc.o base64.o $(EVENT_HANDLER) http_client.o ini.o md5.o misc.o proxy.o sha1.o $(SSL_CLIENT) url.o xmltree.o ftutil.o
 
 CFLAGS += -Wall

lib/misc.c

@@ -306 +306 @@
         for( i = j = 0; t[i]; i ++, j ++ )
         {
-                if( !isalnum( t[i] ) && !strchr( "._-~", t[i] ) )
+                /* if( t[i] <= ' ' || ((unsigned char *)t)[i] >= 128 || t[i] == '%' ) */
+                if( !isalnum( t[i] ) )
                 {
                         sprintf( s + j, "%%%02X", ((unsigned char*)t)[i] );

lib/url.c

@@ -27 +27 @@
 
 /* Convert an URL to a url_t structure */
-int url_set( url_t *url, const char *set_url )
+int url_set( url_t *url, char *set_url )
 {
         char s[MAX_STRING+1];

lib/url.h

@@ -42 +42 @@
 } url_t;
 
-int url_set( url_t *url, const char *set_url );
+int url_set( url_t *url, char *set_url );

lib/xmltree.c

@@ -449 +449 @@
         while( node )
         {
-                char *colon;
-                
-                if( g_strcasecmp( node->name, name ) == 0 ||
-                    ( ( colon = strchr( node->name, ':' ) ) &&
-                      g_strcasecmp( colon + 1, name ) == 0 ) )
+                if( g_strcasecmp( node->name, name ) == 0 )
                         break;
                 
@@ -465 +461 @@
 {
         int i;
-        char *colon;
         
         if( !node )
@@ -473 +468 @@
                 if( g_strcasecmp( node->attr[i].key, key ) == 0 )
                         break;
-        
-        /* This is an awful hack that only takes care of namespace prefixes
-           inside a tag. Since IMHO excessive namespace usage in XMPP is
-           massive overkill anyway (this code exists for almost four years
-           now and never really missed it): Meh. */
-        if( !node->attr[i].key && strcmp( key, "xmlns" ) == 0 &&
-            ( colon = strchr( node->name, ':' ) ) )
-        {
-                *colon = '\0';
-                for( i = 0; node->attr[i].key; i ++ )
-                        if( strncmp( node->attr[i].key, "xmlns:", 6 ) == 0 &&
-                            strcmp( node->attr[i].key + 6, node->name ) == 0 )
-                                break;
-                *colon = ':';
-        }
         
         return node->attr[i].value;

protocols/jabber/conference.c

@@ -272 +272 @@
                 }
                 
-                if( bud != jc->me && bud->flags & JBFLAG_IS_ANONYMOUS )
-                {
-                        /* If JIDs are anonymized, add them to the local
-                           list for the duration of this chat. */
+                if( bud != jc->me )
+                {
                         imcb_add_buddy( ic, bud->ext_jid, NULL );
                         imcb_buddy_nick_hint( ic, bud->ext_jid, bud->resource );

protocols/msn/sb.c

@@ -334 +334 @@
         struct msn_data *md = ic->proto_data;
         
-        if( msn_handler( sb->handler ) != -1 )
-                return TRUE;
-        
-        if( sb->msgq != NULL )
+        if( msn_handler( sb->handler ) == -1 )
         {
                 time_t now = time( NULL );
-                char buf[1024];
                 
                 if( now - md->first_sb_failure > 600 )
@@ -357 +353 @@
                                       "There might be problems delivering your messages." );
                 
-                if( md->msgq == NULL )
-                {
-                        md->msgq = sb->msgq;
-                }
-                else
-                {
-                        GSList *l;
-                        
-                        for( l = md->msgq; l->next; l = l->next );
-                        l->next = sb->msgq;
-                }
-                sb->msgq = NULL;
-                
-                debug( "Moved queued messages back to the main queue, "
-                       "creating a new switchboard to retry." );
-                g_snprintf( buf, sizeof( buf ), "XFR %d SB\r\n", ++md->trId );
-                if( !msn_write( ic, buf, strlen( buf ) ) )
-                        return FALSE;
-        }
-        
-        msn_sb_destroy( sb );
-        return FALSE;
+                if( sb->msgq != NULL )
+                {
+                        char buf[1024];
+                        
+                        if( md->msgq == NULL )
+                        {
+                                md->msgq = sb->msgq;
+                        }
+                        else
+                        {
+                                GSList *l;
+                                
+                                for( l = md->msgq; l->next; l = l->next );
+                                l->next = sb->msgq;
+                        }
+                        sb->msgq = NULL;
+                        
+                        debug( "Moved queued messages back to the main queue, creating a new switchboard to retry." );
+                        g_snprintf( buf, sizeof( buf ), "XFR %d SB\r\n", ++md->trId );
+                        if( !msn_write( ic, buf, strlen( buf ) ) )
+                                return FALSE;
+                }
+                
+                msn_sb_destroy( sb );
+                
+                return FALSE;
+        }
+        else
+        {
+                return TRUE;
+        }
 }
 

protocols/nogaim.c

@@ -665 +665 @@
         u->away = u->status_msg = NULL;
         
-        if( set_getbool( &ic->irc->set, "show_offline" ) && !u->online )
-        {
-                /* always set users as online */
+        if( ( flags & OPT_LOGGED_IN ) && !u->online )
+        {
                 irc_spawn( ic->irc, u );
                 u->online = 1;
-                if( !( flags & OPT_LOGGED_IN ) )
-                {
-                        /* set away message if user isn't really online */
-                        u->away = g_strdup( "User is offline" );
-                }
-        }
-        else if( ( flags & OPT_LOGGED_IN ) && !u->online )
-        {
-                irc_spawn( ic->irc, u );
-                u->online = 1;
         }
         else if( !( flags & OPT_LOGGED_IN ) && u->online )
@@ -685 +674 @@
                 struct groupchat *c;
                 
-                if( set_getbool( &ic->irc->set, "show_offline" ) )
-                {
-                        /* keep offline users in channel and set away message to "offline" */
-                        u->away = g_strdup( "User is offline" );
-
-                        /* Keep showing him/her in the control channel but not in groupchats. */
-                        for( c = ic->groupchats; c; c = c->next )
-                        {
-                                if( remove_chat_buddy_silent( c, handle ) && c->joined )
-                                        irc_part( c->ic->irc, u, c->channel );
-                        }
+                irc_kill( ic->irc, u );
+                u->online = 0;
+                
+                /* Remove him/her from the groupchats to prevent PART messages after he/she QUIT already */
+                for( c = ic->groupchats; c; c = c->next )
+                        remove_chat_buddy_silent( c, handle );
+        }
+        
+        if( flags & OPT_AWAY )
+        {
+                if( state && message )
+                {
+                        u->away = g_strdup_printf( "%s (%s)", state, message );
+                }
+                else if( state )
+                {
+                        u->away = g_strdup( state );
+                }
+                else if( message )
+                {
+                        u->away = g_strdup( message );
                 }
                 else
                 {
-                        /* kill offline users */
-                        irc_kill( ic->irc, u );
-                        u->online = 0;
-
-                        /* Remove him/her from the groupchats to prevent PART messages after he/she QUIT already */
-                        for( c = ic->groupchats; c; c = c->next )
-                                remove_chat_buddy_silent( c, handle );
-                }
-        }
-
-        if( flags & OPT_AWAY )
-        {
-                if( state && message )
-                {
-                        u->away = g_strdup_printf( "%s (%s)", state, message );
-                }
-                else if( state )
-                {
-                        u->away = g_strdup( state );
-                }
-                else if( message )
-                {
-                        u->away = g_strdup( message );
-                }
-                else
-                {
                         u->away = g_strdup( "Away" );
                 }
@@ -733 +706 @@
         }
         
-        /* early if-clause for show_offline even if there is some redundant code here because this isn't LISP but C ;) */
-        if( set_getbool( &ic->irc->set, "show_offline" ) && set_getbool( &ic->irc->set, "away_devoice" ) )
+        /* LISPy... */
+        if( ( set_getbool( &ic->irc->set, "away_devoice" ) ) &&         /* Don't do a thing when user doesn't want it */
+            ( u->online ) &&                                            /* Don't touch offline people */
+            ( ( ( u->online != oo ) && !u->away ) ||                    /* Voice joining people */
+              ( ( u->online == oo ) && ( oa == !u->away ) ) ) )         /* (De)voice people changing state */
         {
                 char *from;
@@ -747 +723 @@
                                                             ic->irc->myhost );
                 }
-
-                /* if we use show_offline, we op online users, voice away users, and devoice/deop offline users */
-                if( flags & OPT_LOGGED_IN )
-                {
-                        /* user is "online" (either really online or away) */
-                        irc_write( ic->irc, ":%s MODE %s %cv%co %s %s", from, ic->irc->channel,
-                                                                  u->away?'+':'-', u->away?'-':'+', u->nick, u->nick );
-                }
-                else
-                {
-                        /* user is offline */
-                        irc_write( ic->irc, ":%s MODE %s -vo %s %s", from, ic->irc->channel, u->nick, u->nick );
-                }
-        }
-        else
-        { 
-                /* LISPy... */
-                if( ( set_getbool( &ic->irc->set, "away_devoice" ) ) &&         /* Don't do a thing when user doesn't want it */
-                    ( u->online ) &&                                            /* Don't touch offline people */
-                    ( ( ( u->online != oo ) && !u->away ) ||                    /* Voice joining people */
-                      ( ( u->online == oo ) && ( oa == !u->away ) ) ) )         /* (De)voice people changing state */
-                {
-                        char *from;
-
-                        if( set_getbool( &ic->irc->set, "simulate_netsplit" ) )
-                        {
-                                from = g_strdup( ic->irc->myhost );
-                        }
-                        else
-                        {
-                                from = g_strdup_printf( "%s!%s@%s", ic->irc->mynick, ic->irc->mynick,
-                                                                    ic->irc->myhost );
-                        }
-                        irc_write( ic->irc, ":%s MODE %s %cv %s", from, ic->irc->channel,
-                                                                  u->away?'-':'+', u->nick );
-                        g_free( from );
-                }
+                irc_write( ic->irc, ":%s MODE %s %cv %s", from, ic->irc->channel,
+                                                          u->away?'-':'+', u->nick );
+                g_free( from );
         }
 }
@@ -1260 +1202 @@
                 return g_strdup_printf( "\x02[\x02\x02\x02%04d-%02d-%02d "
                                         "%02d:%02d:%02d\x02]\x02 ",
-                                        msg.tm_year + 1900, msg.tm_mon + 1, msg.tm_mday,
+                                        msg.tm_year + 1900, msg.tm_mon, msg.tm_mday,
                                         msg.tm_hour, msg.tm_min, msg.tm_sec );
 }

protocols/oscar/oscar.c

@@ -205 +205 @@
 static int gaim_icbm_param_info  (aim_session_t *, aim_frame_t *, ...);
 static int gaim_parse_genericerr (aim_session_t *, aim_frame_t *, ...);
+static int gaim_memrequest       (aim_session_t *, aim_frame_t *, ...);
 static int gaim_selfinfo         (aim_session_t *, aim_frame_t *, ...);
 static int gaim_offlinemsg       (aim_session_t *, aim_frame_t *, ...);
@@ -569 +570 @@
         aim_conn_addhandler(sess, bosconn, AIM_CB_FAM_BUD, AIM_CB_BUD_ERROR, gaim_parse_genericerr, 0);
         aim_conn_addhandler(sess, bosconn, AIM_CB_FAM_BOS, AIM_CB_BOS_ERROR, gaim_parse_genericerr, 0);
+        aim_conn_addhandler(sess, bosconn, AIM_CB_FAM_GEN, 0x1f, gaim_memrequest, 0);
         aim_conn_addhandler(sess, bosconn, AIM_CB_FAM_GEN, AIM_CB_GEN_SELFINFO, gaim_selfinfo, 0);
         aim_conn_addhandler(sess, bosconn, AIM_CB_FAM_ICQ, AIM_CB_ICQ_OFFLINEMSG, gaim_offlinemsg, 0);
@@ -602 +604 @@
 }
 
+struct pieceofcrap {
+        struct im_connection *ic;
+        unsigned long offset;
+        unsigned long len;
+        char *modname;
+        int fd;
+        aim_conn_t *conn;
+        unsigned int inpa;
+};
+
+static gboolean damn_you(gpointer data, gint source, b_input_condition c)
+{
+        struct pieceofcrap *pos = data;
+        struct oscar_data *od = pos->ic->proto_data;
+        char in = '\0';
+        int x = 0;
+        unsigned char m[17];
+
+        while (read(pos->fd, &in, 1) == 1) {
+                if (in == '\n')
+                        x++;
+                else if (in != '\r')
+                        x = 0;
+                if (x == 2)
+                        break;
+                in = '\0';
+        }
+        if (in != '\n') {
+                imcb_error(pos->ic, "Gaim was unable to get a valid hash for logging into AIM."
+                                " You may be disconnected shortly.");
+                b_event_remove(pos->inpa);
+                closesocket(pos->fd);
+                g_free(pos);
+                return FALSE;
+        }
+        /* [WvG] Wheeeee! Who needs error checking anyway? ;-) */
+        read(pos->fd, m, 16);
+        m[16] = '\0';
+        b_event_remove(pos->inpa);
+        closesocket(pos->fd);
+        aim_sendmemblock(od->sess, pos->conn, 0, 16, m, AIM_SENDMEMBLOCK_FLAG_ISHASH);
+        g_free(pos);
+        
+        return FALSE;
+}
+
+static gboolean straight_to_hell(gpointer data, gint source, b_input_condition cond) {
+        struct pieceofcrap *pos = data;
+        char buf[BUF_LONG];
+
+        if (source < 0) {
+                imcb_error(pos->ic, "Gaim was unable to get a valid hash for logging into AIM."
+                                " You may be disconnected shortly.");
+                if (pos->modname)
+                        g_free(pos->modname);
+                g_free(pos);
+                return FALSE;
+        }
+
+        g_snprintf(buf, sizeof(buf), "GET " AIMHASHDATA
+                        "?offset=%ld&len=%ld&modname=%s HTTP/1.0\n\n",
+                        pos->offset, pos->len, pos->modname ? pos->modname : "");
+        write(pos->fd, buf, strlen(buf));
+        if (pos->modname)
+                g_free(pos->modname);
+        pos->inpa = b_input_add(pos->fd, B_EV_IO_READ, damn_you, pos);
+        return FALSE;
+}
+
 /* size of icbmui.ocm, the largest module in AIM 3.5 */
 #define AIM_MAX_FILE_SIZE 98304
+
+int gaim_memrequest(aim_session_t *sess, aim_frame_t *fr, ...) {
+        va_list ap;
+        struct pieceofcrap *pos;
+        guint32 offset, len;
+        char *modname;
+        int fd;
+
+        va_start(ap, fr);
+        offset = (guint32)va_arg(ap, unsigned long);
+        len = (guint32)va_arg(ap, unsigned long);
+        modname = va_arg(ap, char *);
+        va_end(ap);
+
+        if (len == 0) {
+                aim_sendmemblock(sess, fr->conn, offset, len, NULL,
+                                AIM_SENDMEMBLOCK_FLAG_ISREQUEST);
+                return 1;
+        }
+        /* uncomment this when you're convinced it's right. remember, it's been wrong before.
+        if (offset > AIM_MAX_FILE_SIZE || len > AIM_MAX_FILE_SIZE) {
+                char *buf;
+                int i = 8;
+                if (modname)
+                        i += strlen(modname);
+                buf = g_malloc(i);
+                i = 0;
+                if (modname) {
+                        memcpy(buf, modname, strlen(modname));
+                        i += strlen(modname);
+                }
+                buf[i++] = offset & 0xff;
+                buf[i++] = (offset >> 8) & 0xff;
+                buf[i++] = (offset >> 16) & 0xff;
+                buf[i++] = (offset >> 24) & 0xff;
+                buf[i++] = len & 0xff;
+                buf[i++] = (len >> 8) & 0xff;
+                buf[i++] = (len >> 16) & 0xff;
+                buf[i++] = (len >> 24) & 0xff;
+                aim_sendmemblock(sess, command->conn, offset, i, buf, AIM_SENDMEMBLOCK_FLAG_ISREQUEST);
+                g_free(buf);
+                return 1;
+        }
+        */
+
+        pos = g_new0(struct pieceofcrap, 1);
+        pos->ic = sess->aux_data;
+        pos->conn = fr->conn;
+
+        pos->offset = offset;
+        pos->len = len;
+        pos->modname = modname ? g_strdup(modname) : NULL;
+
+        fd = proxy_connect("gaim.sourceforge.net", 80, straight_to_hell, pos);
+        if (fd < 0) {
+                if (pos->modname)
+                        g_free(pos->modname);
+                g_free(pos);
+                imcb_error(sess->aux_data, "Gaim was unable to get a valid hash for logging into AIM."
+                                " You may be disconnected shortly.");
+        }
+        pos->fd = fd;
+
+        return 1;
+}
 
 static int gaim_parse_login(aim_session_t *sess, aim_frame_t *fr, ...) {
@@ -2515 +2651 @@
         char * chatname;
         
-        chatname = g_strdup_printf("%s%s_%d", isdigit(*ic->acc->user) ? "icq_" : "",
-                                   ic->acc->user, chat_id++);
+        chatname = g_strdup_printf("%s%d", ic->acc->user, chat_id++);
   
         ret = oscar_chat_join(ic, chatname, NULL, NULL);

protocols/twitter/twitter.c

@@ -23 +23 @@
 
 #include "nogaim.h"
-#include "oauth.h"
 #include "twitter.h"
 #include "twitter_http.h"
 #include "twitter_lib.h"
+
 
 /**
@@ -50 +50 @@
         return (ic->flags & OPT_LOGGED_IN) == OPT_LOGGED_IN;
 }
-
-static void twitter_main_loop_start( struct im_connection *ic )
-{
-        struct twitter_data *td = ic->proto_data;
-        
-        imcb_log( ic, "Connecting to Twitter" );
-
-        // Run this once. After this queue the main loop function.
-        twitter_main_loop(ic, -1, 0);
-
-        // Queue the main_loop
-        // Save the return value, so we can remove the timeout on logout.
-        td->main_loop_id = b_timeout_add(60000, twitter_main_loop, ic);
-}
-
-
-static const struct oauth_service twitter_oauth =
-{
-        "http://api.twitter.com/oauth/request_token",
-        "http://api.twitter.com/oauth/access_token",
-        "http://api.twitter.com/oauth/authorize",
-        .consumer_key = "xsDNKJuNZYkZyMcu914uEA",
-        .consumer_secret = "FCxqcr0pXKzsF9ajmP57S3VQ8V6Drk4o2QYtqMcOszo",
-};
-
-static gboolean twitter_oauth_callback( struct oauth_info *info );
-
-static void twitter_oauth_start( struct im_connection *ic )
-{
-        struct twitter_data *td = ic->proto_data;
-        
-        imcb_log( ic, "Requesting OAuth request token" );
-
-        td->oauth_info = oauth_request_token( &twitter_oauth, twitter_oauth_callback, ic );
-}
-
-static gboolean twitter_oauth_callback( struct oauth_info *info )
-{
-        struct im_connection *ic = info->data;
-        struct twitter_data *td;
-        
-        if( !g_slist_find( twitter_connections, ic ) )
-                return FALSE;
-        
-        td = ic->proto_data;
-        if( info->stage == OAUTH_REQUEST_TOKEN )
-        {
-                char name[strlen(ic->acc->user)+9], *msg;
-                
-                if( info->request_token == NULL )
-                {
-                        imcb_error( ic, "OAuth error: %s", info->http->status_string );
-                        imc_logout( ic, TRUE );
-                        return FALSE;
-                }
-                
-                sprintf( name, "twitter_%s", ic->acc->user );
-                msg = g_strdup_printf( "To finish OAuth authentication, please visit "
-                                       "%s and respond with the resulting PIN code.",
-                                       info->auth_url );
-                imcb_buddy_msg( ic, name, msg, 0, 0 );
-                g_free( msg );
-        }
-        else if( info->stage == OAUTH_ACCESS_TOKEN )
-        {
-                if( info->token == NULL || info->token_secret == NULL )
-                {
-                        imcb_error( ic, "OAuth error: %s", info->http->status_string );
-                        imc_logout( ic, TRUE );
-                        return FALSE;
-                }
-                
-                /* IM mods didn't do this so far and it's ugly but I should
-                   be able to get away with it... */
-                g_free( ic->acc->pass );
-                ic->acc->pass = oauth_to_string( info );
-                
-                twitter_main_loop_start( ic );
-        }
-        
-        return TRUE;
-}
-
 
 static char *set_eval_mode( set_t *set, char *value )
@@ -144 +61 @@
 }
 
-static gboolean twitter_length_check( struct im_connection *ic, gchar *msg )
-{
-        int max = set_getint( &ic->acc->set, "message_length" ), len;
-        
-        if( max == 0 || ( len = g_utf8_strlen( msg, -1 ) ) <= max )
-                return TRUE;
-        
-        imcb_error( ic, "Maximum message length exceeded: %d > %d", len, max );
-        
-        return FALSE;
-}
-
 static void twitter_init( account_t *acc )
 {
         set_t *s;
-        
-        s = set_add( &acc->set, "message_length", "140", set_eval_int, acc );
         
         s = set_add( &acc->set, "mode", "one", set_eval_mode, acc );
         s->flags |= ACC_SET_OFFLINE_ONLY;
-        
-        s = set_add( &acc->set, "oauth", "true", set_eval_bool, acc );
 }
 
@@ -179 +80 @@
 
         twitter_connections = g_slist_append( twitter_connections, ic );
+
+        td->user = acc->user;
+        td->pass = acc->pass;
+        td->home_timeline_id = 0;
+
         ic->proto_data = td;
-        ic->flags |= OPT_DOES_HTML;
-        
-        td->user = acc->user;
-        if( !set_getbool( &acc->set, "oauth" ) )
-                td->pass = g_strdup( acc->pass );
-        else if( strstr( acc->pass, "oauth_token=" ) )
-                td->oauth_info = oauth_from_string( acc->pass, &twitter_oauth );
-        td->home_timeline_id = 0;
+
+        imcb_log( ic, "Connecting to Twitter" );
+
+        // Run this once. After this queue the main loop function.
+        twitter_main_loop(ic, -1, 0);
+
+        // Queue the main_loop
+        // Save the return value, so we can remove the timeout on logout.
+        td->main_loop_id = b_timeout_add(60000, twitter_main_loop, ic);
         
         sprintf( name, "twitter_%s", acc->user );
         imcb_add_buddy( ic, name, NULL );
         imcb_buddy_status( ic, name, OPT_LOGGED_IN, NULL, NULL );
-        
-        if( td->pass || td->oauth_info )
-                twitter_main_loop_start( ic );
-        else
-                twitter_oauth_start( ic );
 }
 
@@ -217 +119 @@
         if( td )
         {
-                oauth_info_free( td->oauth_info );
-                g_free( td->pass );
                 g_free( td );
         }
@@ -230 +130 @@
 static int twitter_buddy_msg( struct im_connection *ic, char *who, char *message, int away )
 {
-        struct twitter_data *td = ic->proto_data;
-        
         if (g_strncasecmp(who, "twitter_", 8) == 0 &&
             g_strcasecmp(who + 8, ic->acc->user) == 0)
-        {
-                if( set_getbool( &ic->acc->set, "oauth" ) &&
-                    td->oauth_info && td->oauth_info->token == NULL )
-                {
-                        if( !oauth_access_token( message, td->oauth_info ) )
-                        {
-                                imcb_error( ic, "OAuth error: %s", "Failed to send access token request" );
-                                imc_logout( ic, TRUE );
-                                return FALSE;
-                        }
-                }
-                else if( twitter_length_check(ic, message) )
-                        twitter_post_status(ic, message);
-        }
+                twitter_post_status(ic, message);
         else
-        {
                 twitter_direct_messages_new(ic, who, message);
-        }
+        
         return( 0 );
 }
@@ -276 +160 @@
 static void twitter_chat_msg( struct groupchat *c, char *message, int flags )
 {
-        if( c && message && twitter_length_check(c->ic, message))
+        if( c && message )
                 twitter_post_status(c->ic, message);
 }

protocols/twitter/twitter.h

@@ -37 +37 @@
         char* user;
         char* pass;
-        struct oauth_info *oauth_info;
         guint64 home_timeline_id;
         gint main_loop_id;

protocols/twitter/twitter_http.c

@@ -29 +29 @@
 ****************************************************************************/ 
 
+#include "twitter_http.h"
 #include "twitter.h"
 #include "bitlbee.h"
@@ -34 +35 @@
 #include "misc.h"
 #include "base64.h"
-#include "oauth.h"
 #include <ctype.h>
 #include <errno.h>
-
-#include "twitter_http.h"
 
 
@@ -47 +45 @@
  * This is actually pretty generic function... Perhaps it should move to the lib/http_client.c
  */
-void *twitter_http(char *url_string, http_input_function func, gpointer data, int is_post, char* user, char* pass, struct oauth_info* oi, char** arguments, int arguments_len)
+void *twitter_http(char *url_string, http_input_function func, gpointer data, int is_post, char* user, char* pass, char** arguments, int arguments_len)
 {
         url_t *url = g_new0( url_t, 1 );
@@ -112 +110 @@
 
         // If a pass and user are given we append them to the request.
-        if (oi)
-        {
-                char *full_header;
-                
-                full_header = oauth_http_header(oi, is_post ? "POST" : "GET",
-                                                url_string, url_arguments);
-                
-                tmp = g_strdup_printf("%sAuthorization: %s\r\n", request, full_header);
-                g_free(request);
-                g_free(full_header);
-                request = tmp;
-        }
-        else if (userpass_base64)
+        if (userpass_base64)
         {
                 tmp = g_strdup_printf("%sAuthorization: Basic %s\r\n", request, userpass_base64);

protocols/twitter/twitter_http.h

@@ -28 +28 @@
 #include "http_client.h"
 
-struct oauth_info;
-
 void *twitter_http(char *url_string, http_input_function func, gpointer data, int is_post, 
-                   char* user, char* pass, struct oauth_info *oi, char** arguments, int arguments_len);
+                                        char* user, char* pass, char** arguments, int arguments_len);
 
 #endif //_TWITTER_HTTP_H

protocols/twitter/twitter_lib.c

@@ -130 +130 @@
         args[0] = "cursor";
         args[1] = g_strdup_printf ("%d", next_cursor);
-        twitter_http(TWITTER_FRIENDS_IDS_URL, twitter_http_get_friends_ids, ic, 0, td->user, td->pass, td->oauth_info, args, 2);
+        twitter_http(TWITTER_FRIENDS_IDS_URL, twitter_http_get_friends_ids, ic, 0, td->user, td->pass, args, 2);
 
         g_free(args[1]);
@@ -396 +396 @@
         }
 
-        twitter_http(TWITTER_HOME_TIMELINE_URL, twitter_http_get_home_timeline, ic, 0, td->user, td->pass, td->oauth_info, args, td->home_timeline_id ? 4 : 2);
+        twitter_http(TWITTER_HOME_TIMELINE_URL, twitter_http_get_home_timeline, ic, 0, td->user, td->pass, args, td->home_timeline_id ? 4 : 2);
 
         g_free(args[1]);
@@ -510 +510 @@
         {
                 td->http_fails = 0;
-                if (!(ic->flags & OPT_LOGGED_IN))
+                if (!ic->flags & OPT_LOGGED_IN)
                         imcb_connected(ic);
         }
@@ -620 +620 @@
         args[1] = g_strdup_printf ("%d", next_cursor);
 
-        twitter_http(TWITTER_SHOW_FRIENDS_URL, twitter_http_get_statuses_friends, ic, 0, td->user, td->pass, td->oauth_info, args, 2);
+        twitter_http(TWITTER_SHOW_FRIENDS_URL, twitter_http_get_statuses_friends, ic, 0, td->user, td->pass, args, 2);
 
         g_free(args[1]);
@@ -654 +654 @@
         args[0] = "status";
         args[1] = msg;
-        twitter_http(TWITTER_STATUS_UPDATE_URL, twitter_http_post_status, ic, 1, td->user, td->pass, td->oauth_info, args, 2);
+        twitter_http(TWITTER_STATUS_UPDATE_URL, twitter_http_post_status, ic, 1, td->user, td->pass, args, 2);
 //      g_free(args[1]);
 }
@@ -672 +672 @@
         args[3] = msg;
         // Use the same callback as for twitter_post_status, since it does basically the same.
-        twitter_http(TWITTER_DIRECT_MESSAGES_NEW_URL, twitter_http_post_status, ic, 1, td->user, td->pass, td->oauth_info, args, 4);
+        twitter_http(TWITTER_DIRECT_MESSAGES_NEW_URL, twitter_http_post_status, ic, 1, td->user, td->pass, args, 4);
 //      g_free(args[1]);
 //      g_free(args[3]);

protocols/yahoo/yahoo.c

@@ -138 +138 @@
         struct im_connection *ic = imcb_new( acc );
         struct byahoo_data *yd = ic->proto_data = g_new0( struct byahoo_data, 1 );
-        char *s;
         
         yd->logged_in = FALSE;
         yd->current_status = YAHOO_STATUS_AVAILABLE;
-        
-        if( ( s = strchr( acc->user, '@' ) ) && g_strcasecmp( s, "@yahoo.com" ) == 0 )
-                imcb_error( ic, "Your Yahoo! username should just be a username. "
-                                "Do not include any @domain part." );
         
         imcb_log( ic, "Connecting" );
@@ -833 +828 @@
         YList *m;
         
-        if( g_strcasecmp( who, ic->acc->user ) == 0 )
-                /* WTF, Yahoo! seems to echo these now? */
-                return;
-        
         inv = g_malloc( sizeof( struct byahoo_conf_invitation ) );
         memset( inv, 0, sizeof( struct byahoo_conf_invitation ) );