From e3af63f8d941de3e8c26bde5d51352fd3799757f Mon Sep 17 00:00:00 2001 From: Romain Bignon Date: Wed, 22 Oct 2014 15:25:57 +0200 Subject: [PATCH] fix detection of authentication fail --- modules/cmb/module.py | 14 ++++++-------- 1 file changed, 6 insertions(+), 8 deletions(-) diff --git a/modules/cmb/module.py b/modules/cmb/module.py index 3d53e0a0..3b07be75 100644 --- a/modules/cmb/module.py +++ b/modules/cmb/module.py @@ -22,7 +22,7 @@ from weboob.capabilities.bank import Account, Transaction from weboob.tools.backend import Module, BackendConfig from weboob.tools.value import ValueBackendPassword from weboob.capabilities.base import NotAvailable -from weboob.exceptions import BrowserIncorrectPassword, ParseError +from weboob.exceptions import BrowserIncorrectPassword, BrowserHTTPError, ParseError from weboob.browser import Browser from re import match, compile, sub @@ -95,14 +95,12 @@ class CmbModule(Module, CapBank): 'motDePasse': self.config['password'].get() } - response = self.browser.open("https://www.cmb.fr/domiweb/servlet/Identification", allow_redirects=False, data=data) - - if response.status_code == 302: - self.islogged=True - return True - else: + try: + self.browser.open("https://www.cmb.fr/domiweb/servlet/Identification", allow_redirects=False, data=data) + except BrowserHTTPError: raise BrowserIncorrectPassword() - return False + else: + self.islogged=True def iter_accounts(self): if not self.islogged: