Changeset 52b3a99 for protocols

Timestamp:

2005-12-17T18:55:46Z (19 years ago)

Author:

Wilmer van der Gaast <wilmer@…>

Branches:

master

Children:

2db811a

Parents:

8a9afe4 (diff), b5a22e3 (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:

Successfully tested http_client, added support for redirections.

Location:

protocols

Files:

• http_client.c (modified) (8 diffs)
• jabber/Makefile (modified) (2 diffs)
• msn/Makefile (modified) (2 diffs)
• oscar/Makefile (modified) (2 diffs)
• yahoo/Makefile (modified) (2 diffs)

protocols/http_client.c

@@ -5 +5 @@
   \********************************************************************/
 
-/* HTTP(S) module (actually, it only does HTTPS right now)              */
+/* HTTP(S) module                                                       */
 
 /*
@@ -25 +25 @@
 
 #include <string.h>
+#include <stdio.h>
 
 #include "sock.h"
 #include "http_client.h"
+#include "url.h"
 
 
@@ -61 +63 @@
         }
         
+        req->func = func;
+        req->data = data;
         req->request = g_strdup( request );
         req->request_length = strlen( request );
@@ -103 +107 @@
                         if( !sockerr_again() )
                         {
-                                close( req->fd );
+                                closesocket( req->fd );
                                 goto error;
                         }
@@ -188 +192 @@
                 req->reply_headers = g_realloc( req->reply_headers, req->bytes_read + st + 1 );
                 memcpy( req->reply_headers + req->bytes_read, buffer, st );
+                req->bytes_read += st;
         }
         
@@ -208 +213 @@
         if( end2 && end2 < end1 )
         {
-                end1 = end2;
+                end1 = end2 + 1;
                 evil_server = 1;
         }
+        else
+        {
+                end1 += 2;
+        }
         
         if( end1 )
@@ -217 +226 @@
                 
                 if( evil_server )
+                        req->reply_body = end1 + 1;
+                else
                         req->reply_body = end1 + 2;
+        }
+        
+        if( ( end1 = strchr( req->reply_headers, ' ' ) ) != NULL )
+        {
+                if( sscanf( end1 + 1, "%d", &req->status_code ) != 1 )
+                        req->status_code = -1;
+        }
+        else
+        {
+                req->status_code = -1;
+        }
+        
+        if( req->status_code == 301 || req->status_code == 302 )
+        {
+                char *loc, *new_request, *new_host;
+                int error = 0, new_port, new_proto;
+                
+                loc = strstr( req->reply_headers, "\nLocation: " );
+                if( loc == NULL ) /* We can't handle this redirect... */
+                        goto cleanup;
+                
+                loc += 11;
+                while( *loc == ' ' )
+                        loc ++;
+                
+                /* TODO/FIXME: Possibly have to handle relative redirections,
+                   and rewrite Host: headers. Not necessary for now, it's
+                   enough for passport authentication like this. */
+                
+                if( *loc == '/' )
+                {
+                        /* Just a different pathname... */
+                        
+                        /* Since we don't cache the servername, and since we
+                           don't need this yet anyway, I won't implement it. */
+                        
+                        goto cleanup;
+                }
                 else
-                        req->reply_body = end1 + 4;
+                {
+                        /* A whole URL */
+                        url_t *url;
+                        char *s;
+                        
+                        s = strstr( loc, "\r\n" );
+                        if( s == NULL )
+                                goto cleanup;
+                        
+                        url = g_new0( url_t, 1 );
+                        *s = 0;
+                        
+                        if( !url_set( url, loc ) )
+                        {
+                                g_free( url );
+                                goto cleanup;
+                        }
+                        
+                        /* Okay, this isn't fun! We have to rebuild the request... :-( */
+                        new_request = g_malloc( req->request_length + strlen( url->file ) );
+                        
+                        /* So, now I just allocated enough memory, so I'm
+                           going to use strcat(), whether you like it or not. :-) */
+                        
+                        /* First, find the GET/POST/whatever from the original request. */
+                        s = strchr( req->request, ' ' );
+                        if( s == NULL )
+                        {
+                                g_free( new_request );
+                                g_free( url );
+                                goto cleanup;
+                        }
+                        
+                        *s = 0;
+                        sprintf( new_request, "%s %s HTTP/1.0\r\n", req->request, url->file );
+                        *s = ' ';
+                        
+                        s = strstr( req->request, "\r\n" );
+                        if( s == NULL )
+                        {
+                                g_free( new_request );
+                                g_free( url );
+                                goto cleanup;
+                        }
+                        
+                        strcat( new_request, s + 2 );
+                        new_host = g_strdup( url->host );
+                        new_port = url->port;
+                        new_proto = url->proto;
+                        
+                        g_free( url );
+                }
+                
+                if( req->ssl )
+                        ssl_disconnect( req->ssl );
+                else
+                        closesocket( req->fd );
+                
+                req->fd = -1;
+                req->ssl = 0;
+                
+                if( new_proto == PROTO_HTTPS )
+                {
+                        req->ssl = ssl_connect( new_host, new_port, http_ssl_connected, req );
+                        if( req->ssl == NULL )
+                                error = 1;
+                }
+                else
+                {
+                        req->fd = proxy_connect( new_host, new_port, http_connected, req );
+                        if( req->fd < 0 )
+                                error = 1;
+                }
+                
+                if( error )
+                {
+                        g_free( new_request );
+                        goto cleanup;
+                }
+                
+                g_free( req->request );
+                g_free( req->reply_headers );
+                req->request = new_request;
+                req->request_length = strlen( new_request );
+                req->bytes_read = req->bytes_written = req->inpa = 0;
+                req->reply_headers = req->reply_body = NULL;
+                
+                return;
         }
         
@@ -230 +366 @@
                 ssl_disconnect( req->ssl );
         else
-                close( req->fd );
+                closesocket( req->fd );
         
         req->func( req );

protocols/jabber/Makefile

@@ -16 +16 @@
 
 # [SH] Phony targets
-all: jabberr.o
+all: jabber_mod.o
 
 .PHONY: all clean distclean
@@ -33 +33 @@
         @$(CC) -c $(CFLAGS) $< -o $@
 
-jabberr.o: $(objects)
-        @echo '*' Linking jabberr.o
-        @$(LD) $(LFLAGS) $(objects) -o jabberr.o
+jabber_mod.o: $(objects)
+        @echo '*' Linking jabber_mod.o
+        @$(LD) $(LFLAGS) $(objects) -o jabber_mod.o

protocols/msn/Makefile

@@ -16 +16 @@
 
 # [SH] Phony targets
-all: msnn.o
+all: msn_mod.o
         
 .PHONY: all clean distclean
@@ -33 +33 @@
         @$(CC) -c $(CFLAGS) $< -o $@
 
-msnn.o: $(objects)
-        @echo '*' Linking msnn.o
-        @$(LD) $(LFLAGS) $(objects) -o msnn.o
+msn_mod.o: $(objects)
+        @echo '*' Linking msn_mod.o
+        @$(LD) $(LFLAGS) $(objects) -o msn_mod.o
         
 

protocols/oscar/Makefile

@@ -16 +16 @@
 
 # [SH] Phony targets
-all: oscarr.o
+all: oscar_mod.o
 
 .PHONY: all clean distclean
@@ -33 +33 @@
         @$(CC) -c $(CFLAGS) $< -o $@
 
-oscarr.o: $(objects)
-        @echo '*' Linking oscarr.o
-        @$(LD) $(LFLAGS) $(objects) -o oscarr.o
+oscar_mod.o: $(objects)
+        @echo '*' Linking oscar_mod.o
+        @$(LD) $(LFLAGS) $(objects) -o oscar_mod.o

protocols/yahoo/Makefile

@@ -16 +16 @@
 
 # [SH] Phony targets
-all: yahooo.o
+all: yahoo_mod.o
 
 .PHONY: all clean distclean
@@ -33 +33 @@
         @$(CC) -c $(CFLAGS) $< -o $@
 
-yahooo.o: $(objects)
-        @echo '*' Linking yahooo.o
-        @$(LD) $(LFLAGS) $(objects) -o yahooo.o
+yahoo_mod.o: $(objects)
+        @echo '*' Linking yahoo_mod.o
+        @$(LD) $(LFLAGS) $(objects) -o yahoo_mod.o