[evolution-patches] patch for 43387



Here's the reconnect patch.  I've verified it does in fact fix the bug
(i just copied 509 messages from imap to exchange), but it's not clear
to me just how it might affect other operations of libsoup.

Chris
Index: ChangeLog
===================================================================
RCS file: /cvs/gnome/libsoup/ChangeLog,v
retrieving revision 1.307
diff -u -r1.307 ChangeLog
--- ChangeLog	21 May 2003 12:23:10 -0000	1.307
+++ ChangeLog	30 May 2003 21:41:57 -0000
@@ -1,3 +1,12 @@
+2003-05-30  Chris Toshok  <toshok ximian com>
+
+	* libsoup/soup-queue.c (soup_queue_error_cb): always force a
+	reconnect when there's an error with ssl connection.  This fixes
+	#43387, but it runs the risk of sending requests multiple times to
+	the exchange server, and it results in lots of shorter lived
+	connections and more forking (in the ssl proxy case), depending on
+	the length of the operation.
+
 2003-05-21  Dan Winship  <danw ximian com>
 
 	* configure.in: 1.99.22 (codename: French Onion Soup)
Index: libsoup/soup-queue.c
===================================================================
RCS file: /cvs/gnome/libsoup/libsoup/soup-queue.c,v
retrieving revision 1.83
diff -u -r1.83 soup-queue.c
--- libsoup/soup-queue.c	21 May 2003 05:01:51 -0000	1.83
+++ libsoup/soup-queue.c	30 May 2003 21:41:57 -0000
@@ -91,6 +91,8 @@
 					soup_message_issue_callback (req);
 				} else {
 					req->priv->retries++;
+					soup_connection_release (req->connection);
+					req->connection = NULL;
 					soup_message_requeue (req);
 				}
 			} else if (conn_is_new) {


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]