Changes in protocols/twitter/twitter.c [1dd3470:64f8c425]

File:

• protocols/twitter/twitter.c (modified) (13 diffs)

protocols/twitter/twitter.c

@@ -27 +27 @@
 #include "twitter_http.h"
 #include "twitter_lib.h"
+#include "url.h"
 
 /**
@@ -39 +40 @@
                 return 0;
 
-        // If the user uses multiple private message windows we need to get the 
-        // users buddies.
-        if (g_strcasecmp(set_getstr(&ic->acc->set, "mode"), "many") == 0)
-                twitter_get_statuses_friends(ic, -1);
-
         // Do stuff..
         twitter_get_home_timeline(ic, -1);
@@ -55 +51 @@
         struct twitter_data *td = ic->proto_data;
         
-        imcb_log( ic, "Connecting to Twitter" );
+        imcb_log( ic, "Getting initial statuses" );
 
         // Run this once. After this queue the main loop function.
@@ -65 +61 @@
 }
 
+static void twitter_oauth_start( struct im_connection *ic );
+
+void twitter_login_finish( struct im_connection *ic )
+{
+        struct twitter_data *td = ic->proto_data;
+        
+        if( set_getbool( &ic->acc->set, "oauth" ) && !td->oauth_info )
+                twitter_oauth_start( ic );
+        else if( g_strcasecmp( set_getstr( &ic->acc->set, "mode" ), "one" ) != 0 &&
+                 !( td->flags & TWITTER_HAVE_FRIENDS ) )
+        {
+                imcb_log( ic, "Getting contact list" );
+                twitter_get_statuses_friends( ic, -1 );
+        }
+        else
+                twitter_main_loop_start( ic );
+}
 
 static const struct oauth_service twitter_oauth =
@@ -70 +83 @@
         "http://api.twitter.com/oauth/request_token",
         "http://api.twitter.com/oauth/access_token",
-        "http://api.twitter.com/oauth/authorize",
+        "https://api.twitter.com/oauth/authorize",
         .consumer_key = "xsDNKJuNZYkZyMcu914uEA",
         .consumer_secret = "FCxqcr0pXKzsF9ajmP57S3VQ8V6Drk4o2QYtqMcOszo",
@@ -127 +140 @@
                 ic->acc->pass = oauth_to_string( info );
                 
-                twitter_main_loop_start( ic );
+                twitter_login_finish( ic );
         }
         
@@ -160 +173 @@
         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 );
         
@@ -175 +191 @@
 {
         struct im_connection *ic = imcb_new( acc );
-        struct twitter_data *td = g_new0( struct twitter_data, 1 );
+        struct twitter_data *td;
         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;
-        ic->flags |= OPT_DOES_HTML;
+        
+        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( "" );
         
         td->user = acc->user;
-        if( !set_getbool( &acc->set, "oauth" ) )
-                td->pass = g_strdup( acc->pass );
-        else if( strstr( acc->pass, "oauth_token=" ) )
+        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 );
@@ -193 +223 @@
         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 );
+        imcb_log( ic, "Connecting" );
+        
+        twitter_login_finish( ic );
 }
 
@@ -218 +247 @@
         {
                 oauth_info_free( td->oauth_info );
+                g_free( td->url_host );
+                g_free( td->url_path );
                 g_free( td->pass );
                 g_free( td );
@@ -238 +269 @@
                     td->oauth_info && td->oauth_info->token == NULL )
                 {
-                        if( !oauth_access_token( message, td->oauth_info ) )
+                        char pin[strlen(message)+1], *s;
+                        
+                        strcpy( pin, message );
+                        for( s = pin + sizeof( pin ) - 2; s > pin && isspace( *s ); s -- )
+                                *s = '\0';
+                        for( s = pin; *s && isspace( *s ); s ++ ) {}
+                        
+                        if( !oauth_access_token( s, td->oauth_info ) )
                         {
                                 imcb_error( ic, "OAuth error: %s", "Failed to send access token request" );
@@ -268 +306 @@
 static void twitter_add_buddy( struct im_connection *ic, char *who, char *group )
 {
+        twitter_friendships_create_destroy(ic, who, 1);
 }
 
 static void twitter_remove_buddy( struct im_connection *ic, char *who, char *group )
 {
+        twitter_friendships_create_destroy(ic, who, 0);
 }
 
@@ -336 +376 @@
         struct prpl *ret = g_new0(struct prpl, 1);
         
-        ret->options = OPT_NOOTR;
         ret->name = "twitter";
         ret->login = twitter_login;