[Neo-report] r2019 vincent - /trunk/neo/connection.py

nobody at svn.erp5.org nobody at svn.erp5.org
Fri Apr 23 18:36:05 CEST 2010


Author: vincent
Date: Fri Apr 23 18:36:04 2010
New Revision: 2019

Log:
Catch (and reraise) all exceptions instead of just ConnectorException.

Any exception triggered by low-level code should cause a log and
connection closure before being propagated.

Modified:
    trunk/neo/connection.py

Modified: trunk/neo/connection.py
==============================================================================
--- trunk/neo/connection.py [iso-8859-1] (original)
+++ trunk/neo/connection.py [iso-8859-1] Fri Apr 23 18:36:04 2010
@@ -477,7 +477,7 @@
             # should not occurs but it seems it's false
             logging.debug('Connection reset by peer: %r', self.connector)
             self._closure()
-        except ConnectorException:
+        except:
             logging.debug('Unknown connection error: %r', self.connector)
             self._closure()
             # unhandled connector exception
@@ -505,7 +505,7 @@
             # connection resetted by peer
             logging.debug('Connection reset by peer: %r', self.connector)
             self._closure()
-        except ConnectorException:
+        except:
             logging.debug('Unknown connection error: %r', self.connector)
             # unhandled connector exception
             self._closure()





More information about the Neo-report mailing list