plain 0.56.1__tar.gz → 0.57.0__tar.gz

{plain-0.56.1 → plain-0.57.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: plain
-Version: 0.56.1
+Version: 0.57.0
 Summary: A web framework for building products with Python.
 Author-email: Dave Gaeddert <dave.gaeddert@dropseed.dev>
 License-File: LICENSE

{plain-0.56.1 → plain-0.57.0}/plain/CHANGELOG.md

@@ -1,5 +1,20 @@
 # plain changelog
 
+## [0.57.0](https://github.com/dropseed/plain/releases/plain@0.57.0) (2025-08-15)
+
+### What's changed
+
+- The `ResponsePermanentRedirect` class has been removed; use `ResponseRedirect` with `status_code=301` instead ([d5735ea](https://github.com/dropseed/plain/commit/d5735ea4f8))
+- The `RedirectView.permanent` attribute has been replaced with `status_code` for more flexible redirect status codes ([12dda16](https://github.com/dropseed/plain/commit/12dda16731))
+- Updated `RedirectView` initialization parameters: `url_name` replaces `pattern_name`, `preserve_query_params` replaces `query_string`, and removed 410 Gone functionality ([3b9ca71](https://github.com/dropseed/plain/commit/3b9ca713bf))
+
+### Upgrade instructions
+
+- Replace `ResponsePermanentRedirect` imports with `ResponseRedirect` and pass `status_code=301` to the constructor
+- Update `RedirectView` subclasses to use `status_code=301` instead of `permanent=True`
+- Replace `pattern_name` with `url_name` in RedirectView usage
+- Replace `query_string=True` with `preserve_query_params=True` in RedirectView usage
+
 ## [0.56.1](https://github.com/dropseed/plain/releases/plain@0.56.1) (2025-07-30)
 
 ### What's changed

{plain-0.56.1 → plain-0.57.0}/plain/exceptions.py

@@ -53,12 +53,6 @@ class DisallowedHost(SuspiciousOperation):
     pass
 
 
-class DisallowedRedirect(SuspiciousOperation):
-    """Redirect to scheme not in allowed list"""
-
-    pass
-
-
 class TooManyFieldsSent(SuspiciousOperation):
     """
     The number of fields in a GET or POST request exceeded

{plain-0.56.1 → plain-0.57.0}/plain/http/__init__.py

@@ -19,7 +19,6 @@ from plain.http.response import (
     ResponseNotAllowed,
     ResponseNotFound,
     ResponseNotModified,
-    ResponsePermanentRedirect,
     ResponseRedirect,
     ResponseServerError,
     StreamingResponse,
@@ -36,7 +35,6 @@ __all__ = [
     "ResponseBase",
     "StreamingResponse",
     "ResponseRedirect",
-    "ResponsePermanentRedirect",
     "ResponseNotModified",
     "ResponseBadRequest",
     "ResponseForbidden",

{plain-0.56.1 → plain-0.57.0}/plain/http/response.py

@@ -10,10 +10,8 @@ from email.header import Header
 from functools import cached_property
 from http.client import responses
 from http.cookies import SimpleCookie
-from urllib.parse import urlparse
 
 from plain import signals
-from plain.exceptions import DisallowedRedirect
 from plain.http.cookie import sign_cookie_value
 from plain.json import PlainJSONEncoder
 from plain.runtime import settings
@@ -566,19 +564,18 @@ class FileResponse(StreamingResponse):
             self.headers["Content-Disposition"] = content_disposition
 
 
-class ResponseRedirectBase(Response):
-    allowed_schemes = ["http", "https", "ftp"]
+class ResponseRedirect(Response):
+    """HTTP redirect response"""
+
+    status_code = 302
 
     def __init__(self, redirect_to, **kwargs):
         super().__init__(**kwargs)
         self.headers["Location"] = iri_to_uri(redirect_to)
-        parsed = urlparse(str(redirect_to))
-        if parsed.scheme and parsed.scheme not in self.allowed_schemes:
-            raise DisallowedRedirect(
-                f"Unsafe redirect to URL with protocol '{parsed.scheme}'"
-            )
 
-    url = property(lambda self: self.headers["Location"])
+    @property
+    def url(self):
+        return self.headers["Location"]
 
     def __repr__(self):
         return (
@@ -592,18 +589,6 @@ class ResponseRedirectBase(Response):
         )
 
 
-class ResponseRedirect(ResponseRedirectBase):
-    """HTTP 302 response"""
-
-    status_code = 302
-
-
-class ResponsePermanentRedirect(ResponseRedirectBase):
-    """HTTP 301 response"""
-
-    status_code = 301
-
-
 class ResponseNotModified(Response):
     """HTTP 304 response"""
 

{plain-0.56.1 → plain-0.57.0}/plain/internal/middleware/https.py

@@ -1,6 +1,6 @@
 import re
 
-from plain.http import ResponsePermanentRedirect
+from plain.http import ResponseRedirect
 from plain.runtime import settings
 
 
@@ -33,4 +33,6 @@ class HttpsRedirectMiddleware:
             and not any(pattern.search(path) for pattern in self.https_redirect_exempt)
         ):
             host = self.https_redirect_host or request.get_host()
-            return ResponsePermanentRedirect(f"https://{host}{request.get_full_path()}")
+            return ResponseRedirect(
+                f"https://{host}{request.get_full_path()}", status_code=301
+            )

{plain-0.56.1 → plain-0.57.0}/plain/internal/middleware/slash.py

@@ -1,4 +1,4 @@
-from plain.http import ResponsePermanentRedirect
+from plain.http import ResponseRedirect
 from plain.runtime import settings
 from plain.urls import Resolver404, get_resolver
 from plain.utils.http import escape_leading_slashes
@@ -22,7 +22,9 @@ class RedirectSlashMiddleware:
         # If the given URL is "Not Found", then check if we should redirect to
         # a path with a slash appended.
         if response.status_code == 404 and self.should_redirect_with_slash(request):
-            return ResponsePermanentRedirect(self.get_full_path_with_slash(request))
+            return ResponseRedirect(
+                self.get_full_path_with_slash(request), status_code=301
+            )
 
         return response
 

plain-0.57.0/plain/views/redirect.py

@@ -0,0 +1,66 @@
+from plain.http import ResponseRedirect
+from plain.urls import reverse
+
+from .base import View
+
+
+class RedirectView(View):
+    """Provide a redirect on any GET request."""
+
+    status_code = 302
+    url: str | None = None
+    url_name: str | None = None
+    preserve_query_params = False
+
+    def __init__(
+        self, url=None, status_code=None, url_name=None, preserve_query_params=None
+    ):
+        # Allow attributes to be set in RedirectView.as_view(url="...", status_code=301, etc.)
+        self.url = url or self.url
+        self.status_code = status_code if status_code is not None else self.status_code
+        self.url_name = url_name or self.url_name
+        self.preserve_query_params = (
+            preserve_query_params
+            if preserve_query_params is not None
+            else self.preserve_query_params
+        )
+
+    def get_redirect_url(self):
+        """
+        Return the URL redirect to. Keyword arguments from the URL pattern
+        match generating the redirect request are provided as kwargs to this
+        method.
+        """
+        if self.url:
+            url = self.url % self.url_kwargs
+        elif self.url_name:
+            url = reverse(self.url_name, *self.url_args, **self.url_kwargs)
+        else:
+            raise ValueError("RedirectView requires either url or url_name to be set")
+
+        args = self.request.meta.get("QUERY_STRING", "")
+        if args and self.preserve_query_params:
+            url = f"{url}?{args}"
+        return url
+
+    def get(self):
+        url = self.get_redirect_url()
+        return ResponseRedirect(url, status_code=self.status_code)
+
+    def head(self):
+        return self.get()
+
+    def post(self):
+        return self.get()
+
+    def options(self):
+        return self.get()
+
+    def delete(self):
+        return self.get()
+
+    def put(self):
+        return self.get()
+
+    def patch(self):
+        return self.get()

{plain-0.56.1 → plain-0.57.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "plain"
-version = "0.56.1"
+version = "0.57.0"
 description = "A web framework for building products with Python."
 authors = [{name = "Dave Gaeddert", email = "dave.gaeddert@dropseed.dev"}]
 readme = "README.md"

plain-0.56.1/plain/views/redirect.py

@@ -1,77 +0,0 @@
-import logging
-
-from plain.http import (
-    ResponseGone,
-    ResponsePermanentRedirect,
-    ResponseRedirect,
-)
-from plain.urls import reverse
-
-from .base import View
-
-logger = logging.getLogger("plain.request")
-
-
-class RedirectView(View):
-    """Provide a redirect on any GET request."""
-
-    permanent = False
-    url: str | None = None
-    pattern_name: str | None = None
-    query_string = False
-
-    def __init__(self, url=None, permanent=None):
-        # Allow url and permanent to be set in RedirectView.as_view(url="...", permanent=True)
-        self.url = url or self.url
-        self.permanent = permanent if permanent is not None else self.permanent
-
-    def get_redirect_url(self):
-        """
-        Return the URL redirect to. Keyword arguments from the URL pattern
-        match generating the redirect request are provided as kwargs to this
-        method.
-        """
-        if self.url:
-            url = self.url % self.url_kwargs
-        elif self.pattern_name:
-            url = reverse(self.pattern_name, *self.url_args, **self.url_kwargs)
-        else:
-            return None
-
-        args = self.request.meta.get("QUERY_STRING", "")
-        if args and self.query_string:
-            url = f"{url}?{args}"
-        return url
-
-    def get(self):
-        url = self.get_redirect_url()
-        if url:
-            if self.permanent:
-                return ResponsePermanentRedirect(url)
-            else:
-                return ResponseRedirect(url)
-        else:
-            logger.warning(
-                "Gone: %s",
-                self.request.path,
-                extra={"status_code": 410, "request": self.request},
-            )
-            return ResponseGone()
-
-    def head(self):
-        return self.get()
-
-    def post(self):
-        return self.get()
-
-    def options(self):
-        return self.get()
-
-    def delete(self):
-        return self.get()
-
-    def put(self):
-        return self.get()
-
-    def patch(self):
-        return self.get()