From a573f046165aa8f28498021939b40286ab96befe Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Touceda?= <chiiph@leap.se>
Date: Fri, 14 Jun 2013 10:06:46 -0300
Subject: Fix set_eip_status wrong call

---
 src/leap/gui/statuspanel.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/leap/gui/statuspanel.py b/src/leap/gui/statuspanel.py
index 53c19e86..554903d8 100644
--- a/src/leap/gui/statuspanel.py
+++ b/src/leap/gui/statuspanel.py
@@ -189,7 +189,7 @@ class StatusPanelWidget(QtGui.QWidget):
                                                         "it's already "
                                                         "running.")))
         else:
-            self._set_eip_status(status)
+            self.set_eip_status(status)
 
     def set_eip_status_icon(self, status):
         """
-- 
cgit v1.2.3


From d8ccb2845ec34d0d00ef5c1799822da7c1281bea Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Touceda?= <chiiph@leap.se>
Date: Fri, 14 Jun 2013 10:49:41 -0300
Subject: Fix wrong set_status call after refactor

---
 src/leap/gui/mainwindow.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/leap/gui/mainwindow.py b/src/leap/gui/mainwindow.py
index 90e74823..a5905679 100644
--- a/src/leap/gui/mainwindow.py
+++ b/src/leap/gui/mainwindow.py
@@ -1150,7 +1150,8 @@ class MainWindow(QtGui.QMainWindow):
         """
         passed = data[self._provider_bootstrapper.PASSED_KEY]
         if not passed:
-            self._set_status(data[self._provider_bootstrapper.ERROR_KEY])
+            self._login_widget.set_status(
+                data[self._provider_bootstrapper.ERROR_KEY])
             self._already_started_eip = False
 
     def _eip_finished(self, exitCode):
-- 
cgit v1.2.3


From 1b670d268ee26fc06115702aff055884327f85ed Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Touceda?= <chiiph@leap.se>
Date: Fri, 14 Jun 2013 11:08:39 -0300
Subject: Improve error messages in login

---
 src/leap/crypto/srpauth.py | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/leap/crypto/srpauth.py b/src/leap/crypto/srpauth.py
index bcd24de3..d089fa50 100644
--- a/src/leap/crypto/srpauth.py
+++ b/src/leap/crypto/srpauth.py
@@ -459,7 +459,8 @@ class SRPAuth(QtCore.QObject):
         :type failure: twisted.python.failure.Failure
         """
         logger.error("Error logging in %s" % (failure,))
-        self.authentication_finished.emit(False, "%s" % (failure,))
+        self.authentication_finished.emit(False, "%s" % (failure.value,))
+        failure.trap(Exception)
 
     def get_session_id(self):
         return self.__instance.get_session_id()
-- 
cgit v1.2.3


From 8bee5f4e9a1bb0f7069fe41ab37dfec000487d7b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Tom=C3=A1s=20Touceda?= <chiiph@leap.se>
Date: Fri, 14 Jun 2013 12:45:08 -0300
Subject: Actually deferToThread all the things we expect to do in parallel

---
 src/leap/crypto/srpauth.py                | 19 +++++++++++++++----
 src/leap/services/abstractbootstrapper.py |  7 ++++++-
 2 files changed, 21 insertions(+), 5 deletions(-)

diff --git a/src/leap/crypto/srpauth.py b/src/leap/crypto/srpauth.py
index d089fa50..0e95ae64 100644
--- a/src/leap/crypto/srpauth.py
+++ b/src/leap/crypto/srpauth.py
@@ -24,6 +24,7 @@ import json
 
 #this error is raised from requests
 from simplejson.decoder import JSONDecodeError
+from functools import partial
 
 from PySide import QtCore
 from twisted.internet import threads
@@ -321,6 +322,9 @@ class SRPAuth(QtCore.QObject):
 
             self.set_session_id(session_id)
 
+        def _threader(self, cb, res, *args, **kwargs):
+            return threads.deferToThread(cb, res, *args, **kwargs)
+
         def authenticate(self, username, password):
             """
             Executes the whole authentication process for a user
@@ -341,10 +345,17 @@ class SRPAuth(QtCore.QObject):
                                       username=username,
                                       password=password)
 
-            d.addCallback(self._start_authentication, username=username,
-                          password=password)
-            d.addCallback(self._process_challenge, username=username)
-            d.addCallback(self._verify_session)
+            d.addCallback(
+                partial(self._threader,
+                        self._start_authentication),
+                username=username,
+                password=password)
+            d.addCallback(
+                partial(self._threader,
+                        self._process_challenge),
+                username=username)
+            d.addCallback(partial(self._threader,
+                                  self._verify_session))
 
             return d
 
diff --git a/src/leap/services/abstractbootstrapper.py b/src/leap/services/abstractbootstrapper.py
index 2cbd56bc..f0937197 100644
--- a/src/leap/services/abstractbootstrapper.py
+++ b/src/leap/services/abstractbootstrapper.py
@@ -22,6 +22,8 @@ import logging
 
 import requests
 
+from functools import partial
+
 from PySide import QtCore
 from twisted.internet import threads
 from leap.common.check import leap_assert, leap_assert_type
@@ -128,6 +130,9 @@ class AbstractBootstrapper(QtCore.QObject):
             logger.debug("Emitting %s" % (signal,))
             signal.emit({self.PASSED_KEY: True, self.ERROR_KEY: ""})
 
+    def _callback_threader(self, cb, res, *args, **kwargs):
+        return threads.deferToThread(cb, res, *args, **kwargs)
+
     def addCallbackChain(self, callbacks):
         """
         Creates a callback/errback chain on another thread using
@@ -148,7 +153,7 @@ class AbstractBootstrapper(QtCore.QObject):
             if d is None:
                 d = threads.deferToThread(cb)
             else:
-                d.addCallback(cb)
+                d.addCallback(partial(self._callback_threader, cb))
             d.addErrback(self._errback, signal=sig)
             d.addCallback(self._gui_notify, signal=sig)
         d.addErrback(self._gui_errback)
-- 
cgit v1.2.3