we should check for request.user['status'] not request.user['session'], clearly.

diff --git a/mediagoblin/decorators.py b/mediagoblin/decorators.py
index 81b88c9d64ab83f64394305d1eb1f8a4ed94dd2e..1774ce4e71a38b632fa0cf0272aa2d89efb6b1ce 100644 (file)
--- a/mediagoblin/decorators.py
+++ b/mediagoblin/decorators.py
@@ -33,7 +33,7 @@ def require_active_login(controller):
     Require an active login from the user.
     """
     def new_controller_func(request, *args, **kwargs):
-        if not request.user or not request.user.get('session') == 'active':
+        if not request.user or not request.user.get('status') == u'active':
             # TODO: Indicate to the user that they were redirected
             # here because an *active* user is required.
             return exc.HTTPFound(