Decorate mailqueue views with @login_required
authorMagnus Hagander <magnus@hagander.net>
Tue, 8 Oct 2024 12:05:59 +0000 (14:05 +0200)
committerMagnus Hagander <magnus@hagander.net>
Tue, 8 Oct 2024 12:05:59 +0000 (14:05 +0200)
We did explicitly check for superuser permissions, so there was no way
to get in, but without the decorator we wouldn't get the redirect to the
login page.

postgresqleu/mailqueue/backendviews.py

index d79cde463702253d6d97d4d1f7604a7623ec3d70..dbb0faee65326cc9f321a8d5c7fb69938766b4a7 100644 (file)
@@ -1,6 +1,7 @@
 from django.core.exceptions import PermissionDenied
 from django.shortcuts import get_object_or_404
 from django.http import Http404, HttpResponse
+from django.contrib.auth.decorators import login_required
 
 from postgresqleu.util.backendviews import backend_list_editor
 from postgresqleu.mailqueue.backendforms import BackendMailqueueForm
@@ -8,6 +9,7 @@ from postgresqleu.mailqueue.models import QueuedMail
 from postgresqleu.mailqueue.util import parse_mail_content, recursive_parse_attachments_from_message
 
 
+@login_required
 def edit_mailqueue(request, rest):
     if not request.user.is_superuser:
         raise PermissionDenied("Access denied")
@@ -24,6 +26,7 @@ def edit_mailqueue(request, rest):
     )
 
 
+@login_required
 def view_attachment(request, queueid, attname):
     if not request.user.is_superuser:
         raise PermissionDenied("Access denied")