tina4-python 3.13.74__tar.gz → 3.13.76__tar.gz

{tina4_python-3.13.74 → tina4_python-3.13.76}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: tina4-python
-Version: 3.13.74
+Version: 3.13.76
 Summary: Tina4 Python v3 — Zero-dependency, lightweight web framework
 Author-email: Andre van Zuydam <andrevanzuydam@gmail.com>
 License: MIT

{tina4_python-3.13.74 → tina4_python-3.13.76}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "tina4-python"
-version = "3.13.74"
+version = "3.13.76"
 description = "Tina4 Python v3 — Zero-dependency, lightweight web framework"
 authors = [
     {name = "Andre van Zuydam", email = "andrevanzuydam@gmail.com"}

{tina4_python-3.13.74 → tina4_python-3.13.76}/tina4_python/core/server.py

@@ -12,6 +12,7 @@ import asyncio
 import contextvars
 import importlib
 import uuid
+from email.utils import formatdate, parsedate_to_datetime
 from pathlib import Path
 
 from tina4_python.core.request import Request
@@ -1975,6 +1976,25 @@ async def app(scope: dict, receive, send):
         await send({"type": "http.response.body", "body": b""})
         return
 
+    # If-Modified-Since -> 304, for responses carrying a Last-Modified (static
+    # assets). If-None-Match takes precedence (RFC 9110 13.1.3), so this only
+    # runs when the client sent no ETag validator.
+    if not if_none_match:
+        if_modified_since = request.headers.get("if-modified-since", "")
+        last_modified = ""
+        for name, value in headers:
+            if name == b"last-modified":
+                last_modified = value.decode()
+                break
+        if if_modified_since and last_modified:
+            try:
+                if parsedate_to_datetime(last_modified) <= parsedate_to_datetime(if_modified_since):
+                    await send({"type": "http.response.start", "status": 304, "headers": []})
+                    await send({"type": "http.response.body", "body": b""})
+                    return
+            except (TypeError, ValueError):
+                pass  # Unparseable date -> serve the body (never 304 on garbage).
+
     await send({"type": "http.response.start", "status": response.status_code, "headers": headers})
     await send({"type": "http.response.body", "body": response.content})
 
@@ -2010,6 +2030,18 @@ def _try_static(path: str) -> Response | None:
         if file_path.is_file():
             resp = Response()
             resp.file(str(file_path))
+            # Static assets may be cached but must be revalidated on every use,
+            # so a redeployed file reaches the browser on the next load without a
+            # manual hard refresh. The response already carries an ETag
+            # (build_headers) and the pipeline answers If-None-Match with a 304,
+            # so revalidation is a cheap round-trip, not a re-download. A
+            # Last-Modified is added too (pipeline honours If-Modified-Since ->
+            # 304) so the validators match the other frameworks.
+            resp.header("cache-control", "no-cache, must-revalidate")
+            resp.header(
+                "last-modified",
+                formatdate(file_path.stat().st_mtime, usegmt=True),
+            )
             return resp
     return None
 

{tina4_python-3.13.74 → tina4_python-3.13.76}/tina4_python/migration/runner.py

@@ -296,7 +296,11 @@ def _ensure_tracking_table(db, migration_folder: str | None = None):
     # and copy the values across so already-applied migrations are still seen as
     # applied — without this the `migration_name`-based read below finds nothing
     # and every migration re-runs. The old `migration_id` column is left in
-    # place (harmless, ignored from now on).
+    # place: it cannot be dropped portably (SQLite cannot drop a NOT NULL or a
+    # column without a full table rebuild). It is ignored on READS, but it is
+    # NOT ignorable on WRITES — it is NOT NULL on that table, so
+    # `_record_applied` must keep populating it (see there). Leaving it unwritten
+    # is what broke every migration on an upgraded database in 3.13.55-3.13.75.
     if "migration_id" in cols and "migration_name" not in cols:
         col_type = "VARCHAR(500)" if _is_firebird(db) else "TEXT"
         try:
@@ -641,18 +645,36 @@ def _record_applied(db, name: str, desc: str, batch: int, passed: int = 1) -> No
     """
     now = datetime.now(timezone.utc).isoformat()
     db.execute("DELETE FROM tina4_migration WHERE migration_name = ?", [name])
+
+    # Build the column list from the columns that ACTUALLY exist on this table,
+    # so a legacy column an earlier in-place upgrade left behind is populated
+    # rather than defaulted to NULL. A table created by v3 <= 3.13.54 carries
+    # `migration_id NOT NULL UNIQUE`; the rename upgrade adds `migration_name`
+    # beside it but cannot drop the NOT NULL portably (SQLite cannot drop it at
+    # all without a table rebuild), so this insert has to satisfy it or every
+    # migration on that database fails and none can ever apply.
+    cols = _column_names(db)
+    insert_cols = ["migration_name", "description", "batch", "executed_at", "passed"]
+    values = [name, desc, batch, now, passed]
+
+    # v3 <= 3.13.54: `migration_id` held exactly the value `migration_name` holds
+    # now, so writing `name` restores that invariant. Safe against its UNIQUE:
+    # the DELETE above removed this migration's own row first, so a re-run
+    # replaces its row instead of colliding.
+    if "migration_id" in cols:
+        insert_cols.append("migration_id")
+        values.append(name)
+
     if _is_firebird(db):
         row = db.fetch_one("SELECT GEN_ID(GEN_TINA4_MIGRATION_ID, 1) AS next_id FROM RDB$DATABASE")
-        next_id = row["next_id"] if row else 1
-        db.execute(
-            "INSERT INTO tina4_migration (id, migration_name, description, batch, executed_at, passed) VALUES (?, ?, ?, ?, ?, ?)",
-            [next_id, name, desc, batch, now, passed],
-        )
-    else:
-        db.execute(
-            "INSERT INTO tina4_migration (migration_name, description, batch, executed_at, passed) VALUES (?, ?, ?, ?, ?)",
-            [name, desc, batch, now, passed],
-        )
+        insert_cols.insert(0, "id")
+        values.insert(0, row["next_id"] if row else 1)
+
+    placeholders = ", ".join("?" for _ in insert_cols)
+    db.execute(
+        f"INSERT INTO tina4_migration ({', '.join(insert_cols)}) VALUES ({placeholders})",
+        values,
+    )
 
 
 def _migrate(db, migration_folder: str = "migrations", delimiter: str = ";") -> list[str]: