equal
deleted
inserted
replaced
231 if not session.anonymous_session: |
231 if not session.anonymous_session: |
232 self._postlogin(req) |
232 self._postlogin(req) |
233 return session |
233 return session |
234 |
234 |
235 def _update_last_login_time(self, req): |
235 def _update_last_login_time(self, req): |
|
236 # XXX should properly detect missing permission / non writeable source |
|
237 # and avoid "except (RepositoryError, Unauthorized)" below |
|
238 if req.user.metainformation()['source']['adapter'] == 'ldapuser': |
|
239 return |
236 try: |
240 try: |
237 req.execute('SET X last_login_time NOW WHERE X eid %(x)s', |
241 req.execute('SET X last_login_time NOW WHERE X eid %(x)s', |
238 {'x' : req.user.eid}) |
242 {'x' : req.user.eid}) |
239 req.cnx.commit() |
243 req.cnx.commit() |
240 except (RepositoryError, Unauthorized): |
244 except (RepositoryError, Unauthorized): |
241 # ldap user are not writeable for instance |
|
242 req.cnx.rollback() |
245 req.cnx.rollback() |
243 except: |
246 except: |
244 req.cnx.rollback() |
247 req.cnx.rollback() |
245 raise |
248 raise |
246 |
249 |