Bugfix/refresh session (#243)

* Fix redirect when refreshing session

When session expires redirect the user to the page
that they were trying to get to.

* Bump chart version

ap/auth/views/mixins.py

@@ -12,5 +12,6 @@ def dispatch(self, request, *args, **kwargs):
         if not request.user.is_authenticated:
             return redirect(reverse("login"))
         if OIDCSessionValidator(request).expired():
+            request.session["next"] = request.get_full_path()
             return redirect(reverse("login"))
         return super().dispatch(request, *args, **kwargs)

ap/auth/views/views.py

@@ -58,7 +58,8 @@ def get(self, request):
                 return self._login_failure()
             else:
                 self._login_success(request, user, token)
-                return redirect("/")
+                redirect_url = request.session.pop("next", "/")
+                return redirect(redirect_url)
         except OAuthError as error:
             if settings.DEBUG:
                 raise error

ap/views.py

@@ -1,11 +1,23 @@
 from django.http import HttpResponse
+from django.shortcuts import redirect
+from django.urls import reverse
 from django.views import View
 from django.views.generic import TemplateView
 
+from ap.auth.oidc import OIDCSessionValidator
+
 
 class IndexView(TemplateView):
     template_name = "home.html"
 
+    def dispatch(self, request, *args, **kwargs):
+        """
+        If the session has expired, redirect to login
+        """
+        if OIDCSessionValidator(request).expired():
+            return redirect(reverse("login"))
+        return super().dispatch(request, *args, **kwargs)
+
     def get_template_names(self) -> list[str]:
         if not self.request.user.is_authenticated:
             self.template_name = "login.html"

chart/Chart.yaml

@@ -3,8 +3,8 @@ apiVersion: v2
 name: analytical-platform-ui
 description: Analytical Platform UI
 type: application
-version: 0.1.5
-appVersion: 0.1.5
+version: 0.1.6
+appVersion: 0.1.6
 icon: https://upload.wikimedia.org/wikipedia/en/thumb/4/4a/Ministry_of_Justice_logo_%28United_Kingdom%29.svg/611px-Ministry_of_Justice_logo_%28United_Kingdom%29.svg.png
 maintainers:
   - name: moj-data-platform-robot