dbca-utils 2.1.4__tar.gz → 2.2.0__tar.gz

{dbca_utils-2.1.4 → dbca_utils-2.2.0}/PKG-INFO

@@ -1,31 +1,29 @@
 Metadata-Version: 2.4
 Name: dbca-utils
-Version: 2.1.4
+Version: 2.2.0
 Summary: Utilities for DBCA Django apps
 Author-Email: Rocky Chen <rocky.chen@dbca.wa.gov.au>, Ashley Felton <ashley.felton@dbca.wa.gov.au>
 License-Expression: Apache-2.0
 Classifier: Framework :: Django
-Classifier: Framework :: Django :: 4.2
-Classifier: Framework :: Django :: 5.0
 Classifier: Framework :: Django :: 5.2
+Classifier: Framework :: Django :: 6.0
 Classifier: Environment :: Web Environment
 Classifier: Intended Audience :: Developers
 Classifier: Development Status :: 5 - Production/Stable
 Classifier: Programming Language :: Python
 Classifier: Programming Language :: Python :: 3
-Classifier: Programming Language :: Python :: 3.10
-Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Classifier: Programming Language :: Python :: 3.13
+Classifier: Programming Language :: Python :: 3.14
 Classifier: Topic :: Software Development :: Libraries
 Classifier: Topic :: Software Development :: Libraries :: Python Modules
 Project-URL: Homepage, https://github.com/dbca-wa/dbca-utils
 Project-URL: Repository, https://github.com/dbca-wa/dbca-utils.git
-Project-URL: Changelog, https://github.com/dbca-wa/dbca-utils/blob/master/CHANGELOG.md
+Project-URL: Changelog, https://github.com/dbca-wa/dbca-utils/blob/main/CHANGELOG.md
 Project-URL: GitHub, https://github.com/dbca-wa/dbca-utils
-Requires-Python: <4.0,>=3.10
-Requires-Dist: django<6,>=4.2
-Requires-Dist: markupsafe==3.0.3
+Requires-Python: <4.0,>=3.12
+Requires-Dist: django<6.2,>=5.2
+Requires-Dist: markupsafe>=3.0.3
 Description-Content-Type: text/markdown
 
 # Overview
@@ -34,8 +32,8 @@ DBCA Django utility classes and functions.
 
 ## Requirements
 
-- Python 3.10 or later
-- Django 4.2 or later
+- Python 3.12 or later
+- Django 5.2 or later
 
 ## Development
 
@@ -58,10 +56,8 @@ Run unit tests using `pytest` (or `tox`, to test against multiple Python version
 Tagged releases are built and pushed to PyPI automatically using a GitHub
 workflow in the project. Update the project version in `pyproject.toml` and
 tag the required commit with the same value to trigger a release. Packages
-can also be built and uploaded manually, if desired.
-
-Build the project locally using uv, [publish to the PyPI registry](https://docs.astral.sh/uv/guides/publish/#publishing-your-package)
-using the same tool if you require:
+can also be built and uploaded manually to PyPI using [uv](https://docs.astral.sh/uv/guides/publish/#publishing-your-package),
+if required:
 
     uv build
     uv publish

{dbca_utils-2.1.4 → dbca_utils-2.2.0}/README.md

@@ -4,8 +4,8 @@ DBCA Django utility classes and functions.
 
 ## Requirements
 
-- Python 3.10 or later
-- Django 4.2 or later
+- Python 3.12 or later
+- Django 5.2 or later
 
 ## Development
 
@@ -28,10 +28,8 @@ Run unit tests using `pytest` (or `tox`, to test against multiple Python version
 Tagged releases are built and pushed to PyPI automatically using a GitHub
 workflow in the project. Update the project version in `pyproject.toml` and
 tag the required commit with the same value to trigger a release. Packages
-can also be built and uploaded manually, if desired.
-
-Build the project locally using uv, [publish to the PyPI registry](https://docs.astral.sh/uv/guides/publish/#publishing-your-package)
-using the same tool if you require:
+can also be built and uploaded manually to PyPI using [uv](https://docs.astral.sh/uv/guides/publish/#publishing-your-package),
+if required:
 
     uv build
     uv publish

{dbca_utils-2.1.4 → dbca_utils-2.2.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "dbca-utils"
-version = "2.1.4"
+version = "2.2.0"
 description = "Utilities for DBCA Django apps"
 authors = [
     { name = "Rocky Chen", email = "rocky.chen@dbca.wa.gov.au" },
@@ -10,39 +10,37 @@ readme = "README.md"
 license = "Apache-2.0"
 classifiers = [
     "Framework :: Django",
-    "Framework :: Django :: 4.2",
-    "Framework :: Django :: 5.0",
     "Framework :: Django :: 5.2",
+    "Framework :: Django :: 6.0",
     "Environment :: Web Environment",
     "Intended Audience :: Developers",
     "Development Status :: 5 - Production/Stable",
     "Programming Language :: Python",
     "Programming Language :: Python :: 3",
-    "Programming Language :: Python :: 3.10",
-    "Programming Language :: Python :: 3.11",
     "Programming Language :: Python :: 3.12",
     "Programming Language :: Python :: 3.13",
+    "Programming Language :: Python :: 3.14",
     "Topic :: Software Development :: Libraries",
     "Topic :: Software Development :: Libraries :: Python Modules",
 ]
-requires-python = ">=3.10,<4.0"
+requires-python = ">=3.12,<4.0"
 dependencies = [
-    "django>=4.2,<6",
-    "markupsafe==3.0.3",
+    "django>=5.2,<6.2",
+    "markupsafe>=3.0.3",
 ]
 
 [project.urls]
 Homepage = "https://github.com/dbca-wa/dbca-utils"
 Repository = "https://github.com/dbca-wa/dbca-utils.git"
-Changelog = "https://github.com/dbca-wa/dbca-utils/blob/master/CHANGELOG.md"
+Changelog = "https://github.com/dbca-wa/dbca-utils/blob/main/CHANGELOG.md"
 GitHub = "https://github.com/dbca-wa/dbca-utils"
 
 [dependency-groups]
 dev = [
-    "pytest-django>=4.11.1",
+    "pytest-django>=4.12.0",
     "pytest-sugar>=1.1.1",
-    "tox>=4.31.0",
-    "tox-uv>=1.29.0",
+    "tox>=4.53.0",
+    "tox-uv>=1.35.0",
 ]
 
 [build-system]

{dbca_utils-2.1.4 → dbca_utils-2.2.0}/src/dbca_utils/middleware.py

@@ -87,9 +87,9 @@ class SSOLoginMiddleware(MiddlewareMixin):
     """Django middleware to process HTTP requests containing headers set by the Auth2
     SSO service, specificially:
     - `HTTP_REMOTE_USER`
+    - `HTTP_X_EMAIL`
     - `HTTP_X_LAST_NAME`
     - `HTTP_X_FIRST_NAME`
-    - `HTTP_X_EMAIL`
     The middleware assesses requests containing these headers, and (having deferred user
     authentication to the upstream service), retrieves the local Django User and logs
     the user in automatically.
@@ -116,7 +116,7 @@ class SSOLoginMiddleware(MiddlewareMixin):
         # Security check: if the logged-in request user's email does not match the email
         # returned from Auth2, invalidate the current request session and force a new session
         # using the returned SSO values.
-        if request.user.is_authenticated and request.user.email != request.META["HTTP_X_EMAIL"]:
+        if request.user.is_authenticated and request.user.email != request.META.get("HTTP_X_EMAIL", ""):
             logout(request)
 
         # Request user is not authenticated locally: obtain user attributes from the request.META dict
@@ -124,46 +124,57 @@ class SSOLoginMiddleware(MiddlewareMixin):
         if not request.user.is_authenticated:
             attributemap = {
                 "username": "HTTP_REMOTE_USER",
+                "email": "HTTP_X_EMAIL",
                 "last_name": "HTTP_X_LAST_NAME",
                 "first_name": "HTTP_X_FIRST_NAME",
-                "email": "HTTP_X_EMAIL",
             }
+            attributes = {"username": ""}
 
-            for key, value in attributemap.items():
-                if value in request.META:
-                    attributemap[key] = request.META[value]
+            for key, meta_value in attributemap.items():
+                if meta_value in request.META:
+                    attributes[key] = request.META[meta_value]
 
             # Sanitise first_name and last_name values, because end-users have control over these
             # values and could conceivably inject malicious values into them (e.g. a XSS attack).
-            if "first_name" in attributemap:
-                attributemap["first_name"] = strip_tags(attributemap["first_name"])
-                attributemap["first_name"] = str(escape(attributemap["first_name"]))
-            if "last_name" in attributemap:
-                attributemap["last_name"] = strip_tags(attributemap["first_name"])
-                attributemap["last_name"] = str(escape(attributemap["last_name"]))
+            if "first_name" in attributes:
+                attributes["first_name"] = strip_tags(attributes["first_name"])
+                attributes["first_name"] = str(escape(attributes["first_name"]))
+            if "last_name" in attributes:
+                attributes["last_name"] = strip_tags(attributes["last_name"])
+                attributes["last_name"] = str(escape(attributes["last_name"]))
 
             # Optional setting: projects may define accepted user email domains either as
             # a list of strings, or a single string.
             if hasattr(settings, "ALLOWED_EMAIL_SUFFIXES") and settings.ALLOWED_EMAIL_SUFFIXES:
                 if isinstance(settings.ALLOWED_EMAIL_SUFFIXES, str):
-                    allowed_email_suffixes = list(settings.ALLOWED_EMAIL_SUFFIXES)
+                    # If configured as a string, ALLOWED_EMAIL_SUFFIXES must be a comma-separated list (single-item list is OK).
+                    allowed_email_suffixes = settings.ALLOWED_EMAIL_SUFFIXES.split(",")
                 else:
                     allowed_email_suffixes = settings.ALLOWED_EMAIL_SUFFIXES
+                # Validation: allowed_email_suffixes must be a list of strings.
+                if not (isinstance(allowed_email_suffixes, list) and all(isinstance(x, str) for x in allowed_email_suffixes)):
+                    raise ValueError("ALLOWED_EMAIL_SUFFIXES must be a list of strings")
                 # If the user email suffix is not in the allowed list, return a 404 response.
-                if not any([attributemap["email"].lower().endswith(suffix) for suffix in allowed_email_suffixes]):
+                if not any([attributes["email"].lower().endswith(suffix.lower().strip()) for suffix in allowed_email_suffixes]):
                     return http.HttpResponseForbidden()
 
             # Check for an existing User instance.
-            if attributemap["email"] and User.objects.filter(email__iexact=attributemap["email"]).exists():
-                user = User.objects.filter(email__iexact=attributemap["email"])[0]
-            elif User.__name__ != "EmailUser" and User.objects.filter(username__iexact=attributemap["username"]).exists():
-                user = User.objects.filter(username__iexact=attributemap["username"])[0]
+            if attributes["email"] and User.objects.filter(email__iexact=attributes["email"]).exists():
+                user = User.objects.filter(email__iexact=attributes["email"]).first()
+            elif User.__name__ != "EmailUser" and User.objects.filter(username__iexact=attributes["username"]).exists():
+                user = User.objects.filter(username__iexact=attributes["username"]).first()
             else:
                 user = User(last_login=timezone.localtime())
 
             # Set the user's details from the supplied information.
-            user.__dict__.update(attributemap)
-            user.save()
+            user_has_changed = False
+            for attr, value in attributes.items():
+                if getattr(user, attr) != value:
+                    setattr(user, attr, value)
+                    user_has_changed = True
+            if user_has_changed:
+                user.save()
+
             user.backend = "django.contrib.auth.backends.ModelBackend"
 
             # Log the user in.

dbca_utils-2.2.0/tests/templates/tests/test_model_list.html

@@ -0,0 +1,17 @@
+<!doctype html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <title>{{ TITLE }}</title>
+  </head>
+  <body>
+    <div>Authenticated: {{ user.is_authenticated }}</div>
+    <div>Email: {{ user.email }}</div>
+    <div>Name: {{ user.get_full_name }}</div>
+    <ul>
+      {% for object in object_list %}
+      <li>{{ object.name }}, {{ object.created|date:"r" }}</li>
+      {% endfor %}
+    </ul>
+  </body>
+</html>

{dbca_utils-2.1.4 → dbca_utils-2.2.0}/tests/tests.py

@@ -26,7 +26,7 @@ os.environ["TEST_TUPLE"] = "('a', 'b', 'c')"
 os.environ["TEST_BOOL"] = "False"
 
 
-class TestUtils(TestCase):
+class UtilsTest(TestCase):
     def test_env_returns_str(self):
         test_str = env("TEST_STR")
         self.assertTrue(isinstance(test_str, str))
@@ -68,14 +68,12 @@ class TestUtils(TestCase):
         self.assertTrue(isinstance(test_str, str))
 
 
-class TestModelTests(TestCase):
+class ModelTest(TestCase):
     client = Client()
     model = TestModel
 
     def setUp(self):
-        self.user = User.objects.create_user(
-            username="test", email="test@email.com", password="secret"
-        )
+        self.user = User.objects.create_user(username="test", email="test@email.com", password="secret")
         self.test_model = TestModel.objects.create(name=TEST_NAME)
         self.client.login(username="test", password="secret")
 
@@ -109,3 +107,53 @@ class TestModelTests(TestCase):
         self.assertEqual(response.status_code, 200)
         self.assertContains(response, TEST_VAR)
         self.assertContains(response, TEST_NAME)
+
+
+class SSOLoginMiddlewareTest(TestCase):
+    client = Client()
+
+    def setUp(self):
+        self.user = User.objects.create_user(username="testuser", email="testuser@email.com")
+        # Ensure that the client starts logged out.
+        self.client.logout()
+
+    def test_sso_login(self):
+        """Test that requests having the required META headers will automatically sign in users."""
+        url = reverse("test_model_list")
+        response = self.client.get(
+            url,
+            HTTP_REMOTE_USER="testuser",
+            HTTP_X_EMAIL="testuser@email.com",
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, "Authenticated: True")
+
+    def test_user_properties_update(self):
+        """Test that requests with relevant META headers will update user properties."""
+        url = reverse("test_model_list")
+
+        response = self.client.get(
+            url,
+            HTTP_REMOTE_USER="testuser",
+            HTTP_X_EMAIL="testuser@email.com",
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, "Authenticated: True")
+        user = User.objects.get(email="testuser@email.com")
+        self.assertEqual(user.first_name, "")
+        self.assertEqual(user.last_name, "")
+
+        # Simulate a name change via SSO.
+        self.client.logout()
+        response = self.client.get(
+            url,
+            HTTP_REMOTE_USER="testuser",
+            HTTP_X_EMAIL="testuser@email.com",
+            HTTP_X_FIRST_NAME="Test",
+            HTTP_X_LAST_NAME="User",
+        )
+        self.assertEqual(response.status_code, 200)
+        self.assertContains(response, "Name: Test User")
+        user = User.objects.get(email="testuser@email.com")
+        self.assertEqual(user.first_name, "Test")
+        self.assertEqual(user.last_name, "User")

dbca_utils-2.1.4/tests/templates/tests/test_model_list.html

@@ -1,12 +0,0 @@
-<!DOCTYPE html>
-<html lang="en">
-    <head>
-        <meta charset="UTF-8">
-        <title>{{ TITLE }}</title>
-    </head>
-    <body>
-        <ul>
-            {% for object in object_list %}<li>{{ object.name }}, {{ object.created|date:"r" }}</li>{% endfor %}
-        </ul>
-    </body>
-</html>