changeset 24593:9fb7910c1176

Add a debug line and remove a duplicate "login=" get parameter which was causing yahoo's servers to give us an HTTP 400 error, which is technically probably not valid according to http 1.1
author Mark Doliner <mark@kingant.net>
date Sat, 06 Dec 2008 01:38:48 +0000
parents 104f6f755c7e
children 8b429520f6c6 25e2ab1fff1d
files libpurple/protocols/yahoo/yahoo.c
diffstat 1 files changed, 2 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/libpurple/protocols/yahoo/yahoo.c	Sat Dec 06 01:08:20 2008 +0000
+++ b/libpurple/protocols/yahoo/yahoo.c	Sat Dec 06 01:38:48 2008 +0000
@@ -2706,6 +2706,7 @@
 			  strncmp(buf, "HTTP/1.1 302", strlen("HTTP/1.1 302")))) {
 		purple_connection_error_reason(gc, PURPLE_CONNECTION_ERROR_NETWORK_ERROR,
 			_("Received unexpected HTTP response from server."));
+		purple_debug_misc("yahoo", "Unexpected HTTP response: %s\n", buf);
 		return;
 	}
 
@@ -2802,7 +2803,7 @@
 
 static void yahoo_login_page_hash_iter(const char *key, const char *val, GString *url)
 {
-	if (!strcmp(key, "passwd"))
+	if (!strcmp(key, "passwd") || !strcmp(key, "login"))
 		return;
 	g_string_append_c(url, '&');
 	g_string_append(url, key);