Added redirect check for unauthenticated users.

CONTRIBUTORS.txt

@@ -26,6 +26,7 @@ Direct Contributors
 * Kit Sunde
 * Ben Cardy
 * Rag Sagar.V
+* Lacey Williams Henschel 
 
 Other Contributors
 ==================

braces/views/_access.py

@@ -430,8 +430,10 @@ def dispatch(self, request, *args, **kwargs):
         resp = super(RecentLoginRequiredMixin, self).dispatch(
             request, *args, **kwargs)
 
-        delta = datetime.timedelta(seconds=self.max_last_login_delta)
-        if now() > (request.user.last_login + delta):
-            return logout_then_login(request, self.get_login_url())
-        else:
-            return resp
+        if resp.status_code == 200:
+            delta = datetime.timedelta(seconds=self.max_last_login_delta)
+            if now() > (request.user.last_login + delta):
+                return logout_then_login(request, self.get_login_url())
+            else:
+                return resp
+        return resp

docs/changelog.rst

@@ -4,6 +4,7 @@
 Changelog
 =========
 
+* :bug:`176` Added redirect for unauthenticated users. 
 * :support:`-` Changed :ref:`JsonRequestResponseMixin` docs to not use `ugettext_lazy`.
 * :support:`-` Updated tests to include Python 3.2.
 * :bug:`185` Removed `u` prefixes to allow Python 3.2 support.

tests/test_access_mixins.py

@@ -645,3 +645,9 @@ def test_outdated_login(self):
         self.client.login(username=user.username, password='asdf1234')
         resp = self.client.get(self.outdated_view_url)
         assert resp.status_code == 302
+
+    def test_not_logged_in(self):
+        last_login = datetime.datetime.now()
+        user = UserFactory(last_login=last_login)
+        resp = self.client.get(self.recent_view_url)
+        assert resp.status_code != 200