From 8c4870034e20d98f6754abd1c64f2133623e57fd Mon Sep 17 00:00:00 2001 From: shamoon <4887959+shamoon@users.noreply.github.com> Date: Tue, 13 May 2025 09:07:05 -0700 Subject: [PATCH] Fix disable api remote auth test, Django 5.2 no longer uses process_request --- src/paperless/auth.py | 6 +++--- src/paperless/tests/test_remote_user.py | 2 ++ 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/src/paperless/auth.py b/src/paperless/auth.py index 36131847f..c68d63cf0 100644 --- a/src/paperless/auth.py +++ b/src/paperless/auth.py @@ -54,7 +54,7 @@ class HttpRemoteUserMiddleware(PersistentRemoteUserMiddleware): header = settings.HTTP_REMOTE_USER_HEADER_NAME - def process_request(self, request: HttpRequest) -> None: + def __call__(self, request: HttpRequest) -> None: # If remote user auth is enabled only for the frontend, not the API, # then we need dont want to authenticate the user for API requests. if ( @@ -62,8 +62,8 @@ class HttpRemoteUserMiddleware(PersistentRemoteUserMiddleware): and "paperless.auth.PaperlessRemoteUserAuthentication" not in settings.REST_FRAMEWORK["DEFAULT_AUTHENTICATION_CLASSES"] ): - return - return super().process_request(request) + return self.get_response(request) + return super().__call__(request) class PaperlessRemoteUserAuthentication(authentication.RemoteUserAuthentication): diff --git a/src/paperless/tests/test_remote_user.py b/src/paperless/tests/test_remote_user.py index ebe1b3ff5..78b3393d5 100644 --- a/src/paperless/tests/test_remote_user.py +++ b/src/paperless/tests/test_remote_user.py @@ -1,6 +1,7 @@ import os from unittest import mock +from django.conf import settings from django.contrib.auth.models import User from django.test import override_settings from rest_framework import status @@ -91,6 +92,7 @@ class TestRemoteUser(DirectoriesMixin, APITestCase): @override_settings( REST_FRAMEWORK={ + **settings.REST_FRAMEWORK, "DEFAULT_AUTHENTICATION_CLASSES": [ "rest_framework.authentication.BasicAuthentication", "rest_framework.authentication.TokenAuthentication",