Changes in / [ba3233c:7d53efb]

Files:

• bitlbee.h (modified) (1 diff)
• debian/changelog (modified) (1 diff)
• debian/control (modified) (2 diffs)
• debian/rules (modified) (1 diff)
• doc/CHANGES (modified) (1 diff)
• doc/bitlbee.8 (modified) (1 diff)
• doc/user-guide/commands.xml (modified) (1 diff)
• doc/user-guide/quickstart.xml (modified) (2 diffs)
• irc_commands.c (modified) (1 diff)
• lib/misc.c (modified) (2 diffs)
• lib/xmltree.c (modified) (3 diffs)
• protocols/jabber/conference.c (modified) (1 diff)
• protocols/jabber/iq.c (modified) (1 diff)
• protocols/msn/passport.c (modified) (1 diff)
• protocols/msn/sb.c (modified) (2 diffs)
• protocols/oscar/oscar.c (modified) (6 diffs)
• protocols/twitter/twitter.c (modified) (5 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) (14 diffs)
• protocols/twitter/twitter_lib.h (modified) (1 diff)
• protocols/yahoo/yahoo.c (modified) (2 diffs)

bitlbee.h

@@ -35 +35 @@
 
 #define PACKAGE "BitlBee"
-#define BITLBEE_VERSION "1.2.7"
+#define BITLBEE_VERSION "1.2.6a"
 #define VERSION BITLBEE_VERSION
 #define BITLBEE_VER(a,b,c) (((a) << 16) + ((b) << 8) + (c))
-#define BITLBEE_VERSION_CODE BITLBEE_VER(1, 2, 7)
+#define BITLBEE_VERSION_CODE BITLBEE_VER(1, 2, 6)
 
 #define MAX_STRING 511

debian/changelog

@@ -1 @@
-bitlbee (1.2.7-1) unstable; urgency=high
-
-  * New upstream version.
-  * Fixes MSN Messenger login issues, so uploading at high priority.
-
- -- Wilmer van der Gaast <wilmer@gaast.net>  Sat, 15 May 2010 16:15:57 +0100
-
 bitlbee (1.2.6a-1) unstable; urgency=low
 

debian/control

@@ -15 +15 @@
 Description: An IRC to other chat networks gateway
  This program can be used as an IRC server which forwards everything you
- say to people on other chat networks: Jabber, ICQ, AIM, MSN, Yahoo! and
- Twitter.
+ say to people on other chat networks: Jabber, ICQ, AIM, MSN and Yahoo.
 
 Package: bitlbee-dev
@@ -23 +22 @@
 Description: An IRC to other chat networks gateway
  This program can be used as an IRC server which forwards everything you
- say to people on other chat networks: Jabber, ICQ, AIM, MSN, Yahoo! and
- Twitter.
+ say to people on other chat networks: Jabber, ICQ, AIM, MSN and Yahoo.
  .
  This package holds development stuff for compiling plug-ins.

debian/rules

@@ -99 +99 @@
                 find usr -type f -exec md5sum {} \; > DEBIAN/md5sums
 
-ifdef BITLBEE_FORCE_VERSION
         dpkg-gencontrol -ldebian/changelog -isp -pbitlbee-dev -Pdebian/bitlbee-dev -v1:$(BITLBEE_VERSION)-0
-else
-        dpkg-gencontrol -ldebian/changelog -isp -pbitlbee-dev -Pdebian/bitlbee-dev
-endif
 
         dpkg --build debian/bitlbee-dev ..

doc/CHANGES

@@ -3 +3 @@
 
 http://bugs.bitlbee.org/bitlbee/timeline?daysback=90&changeset=on
-
-Version 1.2.7:
-- Fixed problems with MSN Messenger authentication. ("Could not parse
-  Passport server response")
-- Fixed broken typing notifications when talking to GTalk contacts.
-- Fixed an issue with non-anonymous Jabber chatrooms polluting the nick
-  namespace, sometimes generating odd warning messages.
-- Restored ability to start groupchats on ICQ.
-- Added show_offline setting that will also show offline contacts in the
-  control channel.
-- OAuth support for Twitter: This means the module will keep working after
-  June (this also changes "via API" under your tweets into "via BitlBee").
-
-Finished 15 May 2010
 
 Version 1.2.6a:

doc/bitlbee.8

@@ -39 +39 @@
 networks and acts as a gateway. Users can connect to the server
 with any normal IRC client and see their 'buddy list' in
-&bitlbee. It currently supports Oscar (AIM and ICQ),
-MSN, Jabber, Yahoo! and Twitter.
+&bitlbee. BitlBee's protocol support is based on the gaim 
+protocol plugins. BitlBee currently supports Oscar (aim and icq), 
+MSN, Jabber and Yahoo.
 
 \fBbitlbee\fP should be called by

doc/user-guide/commands.xml

@@ -620 +620 @@
 
         <bitlbee-setting name="ignore_auth_requests" type="boolean" scope="account">
-                <default>false</default>
+                <default>true</default>
 
                 <description>

doc/user-guide/quickstart.xml

@@ -3 +3 @@
 
 <para>
-Welcome to BitlBee, your IRC gateway to ICQ, MSN, AOL, Jabber, Yahoo! and Twitter.
+Welcome to BitlBee, your IRC gateway to ICQ, MSN, AOL, Jabber and Yahoo Instant Messaging Systems.
 </para>
 
@@ -43 +43 @@
 
 <para>
-Other available IM protocols are msn, oscar, yahoo and twitter. OSCAR is the protocol used by ICQ and AOL. For more information about the <emphasis>account add</emphasis> command, see <emphasis>help account add</emphasis>.
+Other available IM protocols are msn, oscar, and yahoo. OSCAR is the protocol used by ICQ and AOL. For more information about the <emphasis>account add</emphasis> command, see <emphasis>help account add</emphasis>.
 </para>
 

irc_commands.c

@@ -73 +73 @@
 static void irc_cmd_nick( irc_t *irc, char **cmd )
 {
-        if( irc->status & USTATUS_IDENTIFIED && irc->nick )
-        {
-                irc_reply( irc, 438, "%s %s :You can only change your nick if you're not "
-                           "logged in (i.e. pre-identify)", irc->nick, cmd[1] );
+        if( irc->nick )
+        {
+                irc_reply( irc, 438, ":The hand of the deity is upon thee, thy nick may not change" );
         }
         /* This is not clean, but for now it'll have to be like this... */
-        else if( ( nick_cmp( cmd[1], irc->mynick ) == 0 ) || ( nick_cmp( cmd[1], NS_NICK ) == 0 ) || ( user_find( irc, cmd[1] ) != NULL ) )
-        {
-                irc_reply( irc, 433, "%s :This nick is already in use", cmd[1] );
+        else if( ( nick_cmp( cmd[1], irc->mynick ) == 0 ) || ( nick_cmp( cmd[1], NS_NICK ) == 0 ) )
+        {
+                irc_reply( irc, 433, ":This nick is already in use" );
         }
         else if( !nick_ok( cmd[1] ) )
         {
                 /* [SH] Invalid characters. */
-                irc_reply( irc, 432, "%s :This nick contains invalid characters", cmd[1] );
-        }
-        else if(irc->nick) 
-        {
-                if( user_find( irc, irc->nick ) )
-                        user_rename(irc, irc->nick, cmd[1]);
-
-                irc_write( irc, ":%s!%s@%s NICK %s", irc->nick, irc->user, irc->host, cmd[1] );
-                g_free(irc->nick);
-                irc->nick = g_strdup( cmd[1] );
+                irc_reply( irc, 432, ":This nick contains invalid characters" );
         }
         else

lib/misc.c

@@ -154 +154 @@
 {
         char *start = in;
-        char out[strlen(in)+1];
+        char *out = g_malloc( strlen( in ) + 1 );
         char *s = out, *cs;
         int i, matched;
         
-        memset( out, 0, sizeof( out ) );
+        memset( out, 0, strlen( in ) + 1 );
         
         while( *in )
@@ -220 +220 @@
         
         strcpy( start, out );
+        g_free( out );
 }
 

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/jabber/iq.c

@@ -127 +127 @@
                     !( s = xt_find_attr( c, "xmlns" ) ) )
                 {
+                        imcb_log( ic, "Warning: Received incomplete IQ-%s packet", type );
                         return XT_HANDLED;
                 }

protocols/msn/passport.c

@@ -145 +145 @@
         char *s;
         
-        if( ( s = xt_find_attr( node, "Id" ) ) &&
-            ( strncmp( s, "Compact", 7 ) == 0 ||
-              strncmp( s, "PPToken", 7 ) == 0 ) )
+        if( ( s = xt_find_attr( node, "Id" ) ) && strcmp( s, "PPToken1" ) == 0 )
                 mad->token = g_memdup( node->text, node->text_len + 1 );
         

protocols/msn/sb.c

@@ -328 +328 @@
         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 )
@@ -351 +347 @@
                                       "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/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, GAIM_INPUT_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, ...) {
@@ -1030 +1166 @@
                 m = g_list_append(m, exch);
 
-                g_snprintf(txt, 1024, "Got an invitation to chatroom %s from %s: %s", name, userinfo->sn, args->msg);
+                g_snprintf( txt, 1024, "Got an invitation to chatroom %s from %s: %s", name, userinfo->sn, args->msg );
 
                 inv->ic = ic;
@@ -1036 +1172 @@
                 inv->name = g_strdup(name);
                 
-                imcb_ask(ic, txt, inv, oscar_accept_chat, oscar_reject_chat);
+                imcb_ask( ic, txt, inv, oscar_accept_chat, oscar_reject_chat);
         
                 if (name)
                         g_free(name);
-        } else if (args->reqclass & AIM_CAPS_ICQRTF) {
-                // TODO: constify
-                char text[strlen(args->info.rtfmsg.rtfmsg)+1];
-                strncpy(text, args->info.rtfmsg.rtfmsg, sizeof(text));
-                imcb_buddy_msg(ic, normalize(userinfo->sn), text, 0, 0);
         }
 
@@ -2520 +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

@@ -27 +27 @@
 #include "twitter_http.h"
 #include "twitter_lib.h"
-#include "url.h"
 
 /**
@@ -71 +70 @@
         "http://api.twitter.com/oauth/request_token",
         "http://api.twitter.com/oauth/access_token",
-        "https://api.twitter.com/oauth/authorize",
+        "http://api.twitter.com/oauth/authorize",
         .consumer_key = "xsDNKJuNZYkZyMcu914uEA",
         .consumer_secret = "FCxqcr0pXKzsF9ajmP57S3VQ8V6Drk4o2QYtqMcOszo",
@@ -161 +160 @@
         set_t *s;
         
-        s = set_add( &acc->set, "base_url", TWITTER_API_URL, NULL, acc );
-        s->flags |= ACC_SET_OFFLINE_ONLY;
-        
         s = set_add( &acc->set, "message_length", "140", set_eval_int, acc );
         
@@ -179 +175 @@
 {
         struct im_connection *ic = imcb_new( acc );
-        struct twitter_data *td;
+        struct twitter_data *td = g_new0( struct twitter_data, 1 );
         char name[strlen(acc->user)+9];
-        url_t url;
-
-        if( !url_set( &url, set_getstr( &ic->acc->set, "base_url" ) ) ||
-            ( url.proto != PROTO_HTTP && url.proto != PROTO_HTTPS ) )
-        {
-                imcb_error( ic, "Incorrect API base URL: %s", set_getstr( &ic->acc->set, "base_url" ) );
-                imc_logout( ic, FALSE );
-                return;
-        }
-        
+
         twitter_connections = g_slist_append( twitter_connections, ic );
-        td = g_new0( struct twitter_data, 1 );
         ic->proto_data = td;
-        
-        td->url_ssl = url.proto == PROTO_HTTPS;
-        td->url_port = url.port;
-        td->url_host = g_strdup( url.host );
-        if( strcmp( url.file, "/" ) != 0 )
-                td->url_path = g_strdup( url.file );
-        else
-                td->url_path = g_strdup( "" );
+        ic->flags |= OPT_DOES_HTML;
         
         td->user = acc->user;
-        if( strstr( acc->pass, "oauth_token=" ) )
+        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;
         
         sprintf( name, "twitter_%s", acc->user );
@@ -211 +193 @@
         imcb_buddy_status( ic, name, OPT_LOGGED_IN, NULL, NULL );
         
-        if( td->oauth_info || !set_getbool( &acc->set, "oauth" ) )
+        if( td->pass || td->oauth_info )
                 twitter_main_loop_start( ic );
         else

protocols/twitter/twitter.h

@@ -42 +42 @@
         struct groupchat *home_timeline_gc;
         gint http_fails;
-        
-        gboolean url_ssl;
-        int url_port;
-        char *url_host;
-        char *url_path;
 };
 

protocols/twitter/twitter_http.c

@@ -41 +41 @@
 
 
-static char *twitter_url_append(char *url, char *key, char* value);
+char *twitter_url_append(char *url, char *key, char* value);
 
 /**
@@ -47 +47 @@
  * This is actually pretty generic function... Perhaps it should move to the lib/http_client.c
  */
-void *twitter_http(struct im_connection *ic, char *url_string, http_input_function func, gpointer data, int is_post, char** arguments, int arguments_len)
+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)
 {
-        struct twitter_data *td = ic->proto_data;
+        url_t *url = g_new0( url_t, 1 );
         char *tmp;
-        GString *request = g_string_new("");
+        char *request;
         void *ret;
+        char *userpass = NULL;
+        char *userpass_base64;
         char *url_arguments;
 
-        url_arguments = g_strdup("");
+        // Fill the url structure.
+        if( !url_set( url, url_string ) )
+        {
+                g_free( url );
+                return NULL;
+        }
+
+        if( url->proto != PROTO_HTTP && url->proto != PROTO_HTTPS )
+        {
+                g_free( url );
+                return NULL;
+        }
+
+        // Concatenate user and pass
+        if (user && pass) {
+                userpass = g_strdup_printf("%s:%s", user, pass);
+                userpass_base64 = base64_encode((unsigned char*)userpass, strlen(userpass));
+        } else {
+                userpass_base64 = NULL;
+        }
+
+        url_arguments = g_malloc(1);
+        url_arguments[0] = '\0';
 
         // Construct the url arguments.
@@ -69 +93 @@
         }
 
+        // Do GET stuff...
+        if (!is_post)
+        {
+                // Find the char-pointer of the end of the string.
+                tmp = url->file + strlen(url->file);
+                tmp[0] = '?';
+                // append the url_arguments to the end of the url->file.
+                // TODO GM: Check the length?
+                g_stpcpy (tmp+1, url_arguments);
+        }
+
+
         // Make the request.
-        g_string_printf(request, "%s %s%s%s%s HTTP/1.0\r\n"
-                                 "Host: %s\r\n"
-                                 "User-Agent: BitlBee " BITLBEE_VERSION " " ARCH "/" CPU "\r\n",
-                                 is_post ? "POST" : "GET",
-                                 td->url_path, url_string,
-                                 is_post ? "" : "?", is_post ? "" : url_arguments,
-                                 td->url_host);
+        request = g_strdup_printf(  "%s %s HTTP/1.0\r\n"
+                                    "Host: %s\r\n"
+                                    "User-Agent: BitlBee " BITLBEE_VERSION " " ARCH "/" CPU "\r\n",
+                                    is_post ? "POST" : "GET", url->file, url->host );
 
         // If a pass and user are given we append them to the request.
-        if (td->oauth_info)
+        if (oi)
         {
                 char *full_header;
-                char *full_url;
                 
-                full_url = g_strconcat(set_getstr(&ic->acc->set, "base_url" ), url_string, NULL);
-                full_header = oauth_http_header(td->oauth_info, is_post ? "POST" : "GET",
-                                                full_url, url_arguments);
+                full_header = oauth_http_header(oi, is_post ? "POST" : "GET",
+                                                url_string, url_arguments);
                 
-                g_string_append_printf(request, "Authorization: %s\r\n", full_header);
+                tmp = g_strdup_printf("%sAuthorization: %s\r\n", request, full_header);
+                g_free(request);
                 g_free(full_header);
-                g_free(full_url);
+                request = tmp;
         }
-        else
+        else if (userpass_base64)
         {
-                char userpass[strlen(ic->acc->user)+2+strlen(ic->acc->pass)];
-                char *userpass_base64;
-                
-                g_snprintf(userpass, sizeof(userpass), "%s:%s", ic->acc->user, ic->acc->pass);
-                userpass_base64 = base64_encode((unsigned char*)userpass, strlen(userpass));
-                g_string_append_printf(request, "Authorization: Basic %s\r\n", userpass_base64);
-                g_free( userpass_base64 );
+                tmp = g_strdup_printf("%sAuthorization: Basic %s\r\n", request, userpass_base64);
+                g_free(request);
+                request = tmp;
         }
 
@@ -107 +135 @@
         {
                 // Append the Content-Type and url-encoded arguments.
-                g_string_append_printf(request,
-                                       "Content-Type: application/x-www-form-urlencoded\r\n"
-                                       "Content-Length: %zd\r\n\r\n%s",
-                                       strlen(url_arguments), url_arguments);
+                tmp = g_strdup_printf("%sContent-Type: application/x-www-form-urlencoded\r\nContent-Length: %zd\r\n\r\n%s", 
+                                                                request, strlen(url_arguments), url_arguments);
+                g_free(request);
+                request = tmp;
         } else {
                 // Append an extra \r\n to end the request...
-                g_string_append(request, "\r\n");
+                tmp = g_strdup_printf("%s\r\n", request);
+                g_free(request);
+                request = tmp;
         }
 
-        ret = http_dorequest(td->url_host, td->url_port, td->url_ssl, request->str, func, data);
+        ret = http_dorequest( url->host, url->port,     url->proto == PROTO_HTTPS, request, func, data );
 
+        g_free( url );
+        g_free( userpass );
+        g_free( userpass_base64 );
         g_free( url_arguments );
-        g_string_free( request, TRUE );
+        g_free( request );
         return ret;
 }
 
-static char *twitter_url_append(char *url, char *key, char* value)
+char *twitter_url_append(char *url, char *key, char* value)
 {
         char *key_encoded = g_strndup(key, 3 * strlen(key));

protocols/twitter/twitter_http.h

@@ -30 +30 @@
 struct oauth_info;
 
-void *twitter_http(struct im_connection *ic, char *url_string, http_input_function func,
-                   gpointer data, int is_post, char** arguments, int arguments_len);
+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);
 
 #endif //_TWITTER_HTTP_H

protocols/twitter/twitter_lib.c

@@ -117 +117 @@
 }
 
-/* Warning: May return a malloc()ed value, which will be free()d on the next
-   call. Only for short-term use. */
-static char *twitter_parse_error(struct http_request *req)
-{
-        static char *ret = NULL;
-        struct xt_parser *xp = NULL;
-        struct xt_node *node;
-        
-        g_free(ret);
-        ret = NULL;
-        
-        if (req->body_size > 0)
-        {
-                xp = xt_new(NULL, NULL);
-                xt_feed(xp, req->reply_body, req->body_size);
-                
-                if ((node = xt_find_node(xp->root, "hash")) &&
-                    (node = xt_find_node(node->children, "error")) &&
-                    node->text_len > 0)
-                {
-                        ret = g_strdup_printf("%s (%s)", req->status_string, node->text);
-                        xt_free(xp);
-                        return ret;
-                }
-                
-                xt_free(xp);
-        }
-        
-        return req->status_string;
-}
-
 static void twitter_http_get_friends_ids(struct http_request *req);
 
@@ -155 +124 @@
 void twitter_get_friends_ids(struct im_connection *ic, int next_cursor)
 {
+        struct twitter_data *td = ic->proto_data;
+
         // Primitive, but hey! It works...      
         char* args[2];
         args[0] = "cursor";
         args[1] = g_strdup_printf ("%d", next_cursor);
-        twitter_http(ic, TWITTER_FRIENDS_IDS_URL, twitter_http_get_friends_ids, ic, 0, args, 2);
+        twitter_http(TWITTER_FRIENDS_IDS_URL, twitter_http_get_friends_ids, ic, 0, td->user, td->pass, td->oauth_info, args, 2);
 
         g_free(args[1]);
@@ -225 +196 @@
                 // It didn't go well, output the error and return.
                 if (++td->http_fails >= 5)
-                        imcb_error(ic, "Could not retrieve friends: %s", twitter_parse_error(req));
+                        imcb_error(ic, "Could not retrieve friends. HTTP STATUS: %d", req->status_code);
                 
                 return;
@@ -425 +396 @@
         }
 
-        twitter_http(ic, TWITTER_HOME_TIMELINE_URL, twitter_http_get_home_timeline, ic, 0, args, td->home_timeline_id ? 4 : 2);
+        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);
 
         g_free(args[1]);
@@ -463 +434 @@
                 twitter_add_buddy(ic, status->user->screen_name, status->user->name);
                 
-                strip_html(status->text);
-                
                 // Say it!
                 if (g_strcasecmp(td->user, status->user->screen_name) == 0)
@@ -502 +471 @@
                 status = l->data;
                 
-                strip_html( status->text );
                 if( mode_one )
                         text = g_strdup_printf( "\002<\002%s\002>\002 %s",
@@ -555 +523 @@
                 // It didn't go well, output the error and return.
                 if (++td->http_fails >= 5)
-                        imcb_error(ic, "Could not retrieve " TWITTER_HOME_TIMELINE_URL ": %s", twitter_parse_error(req));
+                        imcb_error(ic, "Could not retrieve " TWITTER_HOME_TIMELINE_URL ". HTTP STATUS: %d", req->status_code);
                 
                 return;
@@ -607 +575 @@
                 // It didn't go well, output the error and return.
                 if (++td->http_fails >= 5)
-                        imcb_error(ic, "Could not retrieve " TWITTER_SHOW_FRIENDS_URL ": %s", twitter_parse_error(req));
+                        imcb_error(ic, "Could not retrieve " TWITTER_SHOW_FRIENDS_URL " HTTP STATUS: %d", req->status_code);
                 
                 return;
@@ -646 +614 @@
 void twitter_get_statuses_friends(struct im_connection *ic, int next_cursor)
 {
+        struct twitter_data *td = ic->proto_data;
+
         char* args[2];
         args[0] = "cursor";
         args[1] = g_strdup_printf ("%d", next_cursor);
 
-        twitter_http(ic, TWITTER_SHOW_FRIENDS_URL, twitter_http_get_statuses_friends, ic, 0, args, 2);
+        twitter_http(TWITTER_SHOW_FRIENDS_URL, twitter_http_get_statuses_friends, ic, 0, td->user, td->pass, td->oauth_info, args, 2);
 
         g_free(args[1]);
@@ -669 +639 @@
         if (req->status_code != 200) {
                 // It didn't go well, output the error and return.
-                imcb_error(ic, "HTTP error: %s", twitter_parse_error(req));
+                imcb_error(ic, "HTTP Error... STATUS: %d", req->status_code);
                 return;
         }
@@ -679 +649 @@
 void twitter_post_status(struct im_connection *ic, char* msg)
 {
+        struct twitter_data *td = ic->proto_data;
+
         char* args[2];
         args[0] = "status";
         args[1] = msg;
-        twitter_http(ic, TWITTER_STATUS_UPDATE_URL, twitter_http_post, ic, 1, args, 2);
+        twitter_http(TWITTER_STATUS_UPDATE_URL, twitter_http_post, ic, 1, td->user, td->pass, td->oauth_info, args, 2);
 //      g_free(args[1]);
 }
@@ -692 +664 @@
 void twitter_direct_messages_new(struct im_connection *ic, char *who, char *msg)
 {
+        struct twitter_data *td = ic->proto_data;
+
         char* args[4];
         args[0] = "screen_name";
@@ -698 +672 @@
         args[3] = msg;
         // Use the same callback as for twitter_post_status, since it does basically the same.
-        twitter_http(ic, TWITTER_DIRECT_MESSAGES_NEW_URL, twitter_http_post, ic, 1, args, 4);
+        twitter_http(TWITTER_DIRECT_MESSAGES_NEW_URL, twitter_http_post, ic, 1, td->user, td->pass, td->oauth_info, args, 4);
 //      g_free(args[1]);
 //      g_free(args[3]);
@@ -705 +679 @@
 void twitter_friendships_create_destroy(struct im_connection *ic, char *who, int create)
 {
+        struct twitter_data *td = ic->proto_data;
+
         char* args[2];
         args[0] = "screen_name";
         args[1] = who;
-        twitter_http(ic, create ? TWITTER_FRIENDSHIPS_CREATE_URL : TWITTER_FRIENDSHIPS_DESTROY_URL, twitter_http_post, ic, 1, args, 2);
-}
+        twitter_http(create ? TWITTER_FRIENDSHIPS_CREATE_URL : TWITTER_FRIENDSHIPS_DESTROY_URL, twitter_http_post, ic, 1, td->user, td->pass, td->oauth_info, args, 2);
+}
+
+
+

protocols/twitter/twitter_lib.h

@@ -32 +32 @@
 
 /* Status URLs */
-#define TWITTER_STATUS_UPDATE_URL "/statuses/update.xml"
-#define TWITTER_STATUS_SHOW_URL "/statuses/show/"
-#define TWITTER_STATUS_DESTROY_URL "/statuses/destroy/"
+#define TWITTER_STATUS_UPDATE_URL TWITTER_API_URL "/statuses/update.xml"
+#define TWITTER_STATUS_SHOW_URL TWITTER_API_URL "/statuses/show/"
+#define TWITTER_STATUS_DESTROY_URL TWITTER_API_URL "/statuses/destroy/"
 
 /* Timeline URLs */
-#define TWITTER_PUBLIC_TIMELINE_URL "/statuses/public_timeline.xml"
-#define TWITTER_FEATURED_USERS_URL "/statuses/featured.xml"
-#define TWITTER_FRIENDS_TIMELINE_URL "/statuses/friends_timeline.xml"
-#define TWITTER_HOME_TIMELINE_URL "/statuses/home_timeline.xml"
-#define TWITTER_MENTIONS_URL "/statuses/mentions.xml"
-#define TWITTER_USER_TIMELINE_URL "/statuses/user_timeline.xml"
+#define TWITTER_PUBLIC_TIMELINE_URL TWITTER_API_URL "/statuses/public_timeline.xml"
+#define TWITTER_FEATURED_USERS_URL TWITTER_API_URL "/statuses/featured.xml"
+#define TWITTER_FRIENDS_TIMELINE_URL TWITTER_API_URL "/statuses/friends_timeline.xml"
+#define TWITTER_HOME_TIMELINE_URL TWITTER_API_URL "/statuses/home_timeline.xml"
+#define TWITTER_MENTIONS_URL TWITTER_API_URL "/statuses/mentions.xml"
+#define TWITTER_USER_TIMELINE_URL TWITTER_API_URL "/statuses/user_timeline.xml"
 
 /* Users URLs */
-#define TWITTER_SHOW_USERS_URL "/users/show.xml"
-#define TWITTER_SHOW_FRIENDS_URL "/statuses/friends.xml"
-#define TWITTER_SHOW_FOLLOWERS_URL "/statuses/followers.xml"
+#define TWITTER_SHOW_USERS_URL TWITTER_API_URL "/users/show.xml"
+#define TWITTER_SHOW_FRIENDS_URL TWITTER_API_URL "/statuses/friends.xml"
+#define TWITTER_SHOW_FOLLOWERS_URL TWITTER_API_URL "/statuses/followers.xml"
 
 /* Direct messages URLs */
-#define TWITTER_DIRECT_MESSAGES_URL "/direct_messages.xml"
-#define TWITTER_DIRECT_MESSAGES_NEW_URL "/direct_messages/new.xml"
-#define TWITTER_DIRECT_MESSAGES_SENT_URL "/direct_messages/sent.xml"
-#define TWITTER_DIRECT_MESSAGES_DESTROY_URL "/direct_messages/destroy/"
+#define TWITTER_DIRECT_MESSAGES_URL TWITTER_API_URL "/direct_messages.xml"
+#define TWITTER_DIRECT_MESSAGES_NEW_URL TWITTER_API_URL "/direct_messages/new.xml"
+#define TWITTER_DIRECT_MESSAGES_SENT_URL TWITTER_API_URL "/direct_messages/sent.xml"
+#define TWITTER_DIRECT_MESSAGES_DESTROY_URL TWITTER_API_URL "/direct_messages/destroy/"
 
 /* Friendships URLs */
-#define TWITTER_FRIENDSHIPS_CREATE_URL "/friendships/create.xml"
-#define TWITTER_FRIENDSHIPS_DESTROY_URL "/friendships/destroy.xml"
-#define TWITTER_FRIENDSHIPS_SHOW_URL "/friendships/show.xml"
+#define TWITTER_FRIENDSHIPS_CREATE_URL TWITTER_API_URL "/friendships/create.xml"
+#define TWITTER_FRIENDSHIPS_DESTROY_URL TWITTER_API_URL "/friendships/destroy.xml"
+#define TWITTER_FRIENDSHIPS_SHOW_URL TWITTER_API_URL "/friendships/show.xml"
 
 /* Social graphs URLs */
-#define TWITTER_FRIENDS_IDS_URL "/friends/ids.xml"
-#define TWITTER_FOLLOWERS_IDS_URL "/followers/ids.xml"
+#define TWITTER_FRIENDS_IDS_URL TWITTER_API_URL "/friends/ids.xml"
+#define TWITTER_FOLLOWERS_IDS_URL TWITTER_API_URL "/followers/ids.xml"
 
 /* Account URLs */
-#define TWITTER_ACCOUNT_RATE_LIMIT_URL "/account/rate_limit_status.xml"
+#define TWITTER_ACCOUNT_RATE_LIMIT_URL TWITTER_API_URL "/account/rate_limit_status.xml"
 
 /* Favorites URLs */
-#define TWITTER_FAVORITES_GET_URL "/favorites.xml"
-#define TWITTER_FAVORITE_CREATE_URL "/favorites/create/"
-#define TWITTER_FAVORITE_DESTROY_URL "/favorites/destroy/"
+#define TWITTER_FAVORITES_GET_URL TWITTER_API_URL "/favorites.xml"
+#define TWITTER_FAVORITE_CREATE_URL TWITTER_API_URL "/favorites/create/"
+#define TWITTER_FAVORITE_DESTROY_URL TWITTER_API_URL "/favorites/destroy/"
 
 /* Block URLs */
-#define TWITTER_BLOCKS_CREATE_URL "/blocks/create/"
-#define TWITTER_BLOCKS_DESTROY_URL "/blocks/destroy/"
+#define TWITTER_BLOCKS_CREATE_URL TWITTER_API_URL "/blocks/create/"
+#define TWITTER_BLOCKS_DESTROY_URL TWITTER_API_URL "/blocks/destroy/"
 
 void twitter_get_friends_ids(struct im_connection *ic, int next_cursor);

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 ) );