Use class-based views rather than functions

This commit is contained in:
Martin Pépin 2020-05-12 22:35:20 +02:00
parent add3b56690
commit e3c3513a21
3 changed files with 70 additions and 50 deletions

View file

@ -7,10 +7,10 @@
<title>ENS Auth</title>
</head>
<body>
<p><a href="{% url "authens:login.cas" %}?next={{ next | urlencode }}">
<p><a href="{% url "authens:login.cas" %}?next={{ next_page | urlencode }}">
{% trans "Login par CAS" %}
</a></p>
<p><a href="{% url "authens:login.pwd" %}?next={{ next | urlencode }}">
<p><a href="{% url "authens:login.pwd" %}?next={{ next_page | urlencode }}">
{% trans "Login par mot de passe" %}
</a></p>
</body>

View file

@ -5,8 +5,8 @@ from authens import views
app_name = "authens"
urlpatterns = [
path("login/choose", views.login_switch, name="login"),
path("login/cas", views.cas_login, name="login.cas"),
path("login/pwd", views.pwd_login, name="login.pwd"),
path("login/choose", views.LoginSwitchView.as_view(), name="login"),
path("login/cas", views.CASLoginView.as_view(), name="login.cas"),
path("login/pwd", views.PasswordLoginView.as_view(), name="login.pwd"),
path("logout", auth_views.LogoutView.as_view(), name="logout"),
]

View file

@ -1,14 +1,15 @@
from django.conf import settings
from django.contrib import auth
from django.core.exceptions import PermissionDenied
from django.views.decorators.http import require_GET
from django.shortcuts import redirect, render
from django.views.generic import RedirectView, TemplateView
from django.shortcuts import redirect
from django.utils.translation import gettext_lazy as _
from authens.utils import get_cas_client
def _get_next_url(request):
class NextPageMixin:
def next_page_url(self):
"""Decide where to go after a successful login.
Look for (in order):
@ -16,6 +17,7 @@ def _get_next_url(request):
- a `CASNEXT` session variable;
- the `LOGIN_REDIRECT_URL` django setting.
"""
request = self.request
next_page = request.GET.get("next")
if next_page is None and "CASNEXT" in request.session:
next_page = request.session["CASNEXT"]
@ -25,33 +27,51 @@ def _get_next_url(request):
return next_page
@require_GET
def login_switch(request):
next_page = _get_next_url(request)
class LoginSwitchView(NextPageMixin, TemplateView):
"""Simple page letting the user choose between password and CAS authentication."""
template_name = "authens/login_switch.html"
http_method_names = ["get"]
def get(self, request, *args, **kwargs):
if request.user.is_authenticated:
return redirect(next_page)
return render(request, "authens/login_switch.html", {"next": next_page})
return redirect(self.next_page_url())
return super().get(request, *args, **kwargs)
def get_context_data(self, **kwargs):
ctx = super().get_context_data(**kwargs)
ctx["next_page"] = self.next_page_url()
return ctx
@require_GET
def cas_login(request):
next_page = _get_next_url(request)
class CASLoginView(NextPageMixin, RedirectView):
"""CAS authentication view.
Implement the CAS authentication scheme:
1. We first redirect the user to the student CAS.
2. The user comes back with a ticket, we validate it to make sure the user is legit
(validation is delegated to the ENSCASBackend).
3. We redirect the user to the next page.
"""
http_method_names = ["get"]
def get_redirect_url(self, *args, **kwargs):
request = self.request
ticket = request.GET.get("ticket")
# User's request: redirect the user to cas.eleves
if not ticket:
request.session["CASNEXT"] = next_page # remember next_page
request.session["CASNEXT"] = self.next_page_url()
cas_client = get_cas_client(request)
return redirect(cas_client.get_login_url())
return cas_client.get_login_url()
# CAS' request: validate the ticket
user = auth.authenticate(request, ticket=ticket)
if user is None:
raise PermissionDenied(_("Connection échouée !"))
# Success: log the user in
auth.login(request, user)
return redirect(next_page)
return self.next_page_url()
pwd_login = auth.views.LoginView.as_view(template_name="authens/pwd_login.html")
class PasswordLoginView(auth.views.LoginView):
template_name = "authens/pwd_login.html"