web/views/authentication.py
branchtls-sprint
changeset 1690 e4f7d2ddc99a
parent 1668 d2ac1d681d70
parent 1664 03ebeccf9f1d
child 1977 606923dff11b
--- a/web/views/authentication.py	Tue May 05 17:13:30 2009 +0200
+++ b/web/views/authentication.py	Tue May 05 17:13:37 2009 +0200
@@ -53,11 +53,15 @@
         return cnx
 
     def login_from_email(self, login):
+        # XXX should not be called from web interface
         session = self.repo.internal_session()
-        rset = session.execute('Any L WHERE U login L, U primary_email M, '
-                               'M address %(login)s', {'login': login})
-        if rset.rowcount == 1:
-            login = rset[0][0]
+        try:
+            rset = session.execute('Any L WHERE U login L, U primary_email M, '
+                                   'M address %(login)s', {'login': login})
+            if rset.rowcount == 1:
+                login = rset[0][0]
+        finally:
+            session.close()
         return login
 
     def authenticate(self, req, _login=None, _password=None):