Browse Source

HUE-6238 [useradmin] ConcurrentUserSessionMiddleware - missing user

jdesjean 8 years ago
parent
commit
704d710
1 changed files with 6 additions and 1 deletions
  1. 6 1
      apps/useradmin/src/useradmin/middleware.py

+ 6 - 1
apps/useradmin/src/useradmin/middleware.py

@@ -109,7 +109,12 @@ class ConcurrentUserSessionMiddleware(object):
   Middleware that remove concurrent user sessions when configured
   """
   def process_response(self, request, response):
-    if request.user.is_authenticated() and request.session.modified and request.user.id:
+    try:
+      user = request.user
+    except AttributeError: # When the request does not store user. We care only about the login request which does store the user.
+      return response
+
+    if request.user.is_authenticated() and request.session.modified and request.user.id: # request.session.modified checks if a user just logged in
       limit = SESSION.CONCURRENT_USER_SESSION_LIMIT.get()
       if limit:
         count = 1;