refactoring-backend #4

lukas.pupkalipinski · 2026-05-24T22:05:22+02:00

lukas.pupkalipinski commented

2026-05-24 22:05:22 +02:00

No description provided.

lukas.pupkalipinski added 14 commits 2026-05-24 22:05:22 +02:00

chore: bump to v0.9.19-rc.1 and add local OpenAPI build support 12fe70d768

- Add release candidate (rc) support to release.sh with latestRC tagging
- Bump VERSION, backend pyproject.toml, and frontend package.json to 0.9.19-rc.1
- Add local frontend/openapi.json so build no longer needs running backend
- Update generate:types and validate-types.sh to use local openapi.json
- Fix frontend tests: remove unused imports/variables and update mock data

chore: release v0.9.19-rc.2 dcee222a41

fix(auth): suppress misleading 502 warning during session validation 2e5ac092bf

A 502 Bad Gateway is a server/gateway error, not a network error.
Logging it as a 'Session validation network error' is noisy and
misleading during startup when nginx is temporarily unreachable.

Silently skip the console.warn for 5xx errors in handleValidationError
while keeping the warning for actual network errors.

chore: release v0.9.19-rc.3 9d2d6fadf3

fix(db): fix migration failures when upgrading from 0.8.0 schema 9fe52755a5

Migration 1: remove idx_sessions_token_hash from _SCHEMA_STATEMENTS.
The legacy schema has sessions.token (not token_hash). The IF NOT EXISTS
guard only prevents duplicate index names — it still requires the column
to exist. Migration 2 drops and rebuilds sessions with token_hash anyway,
so creating the index in migration 1 was redundant.

Migration 3: replace ALTER TABLE ADD COLUMN with a table rebuild.
SQLite rejects ALTER TABLE ADD COLUMN NOT NULL DEFAULT <expression> when
the table already contains rows. The old DB has ~181k geo_cache rows, so
the ALTER always failed. Rebuild copies existing rows with last_seen set
to cached_at as a reasonable approximation of last-seen time.

chore: release v0.9.19-rc.4 99e1b74405

chore(release): bump version to 0.9.19-rc.4 aebe0d0236

- Add production Docker Compose configuration

- Add check_auth.py diagnostic script for session 401 debugging

chore: release v0.9.19-rc.5 5a12d1c22f

docs: add comprehensive task backlog and bump version to rc.5 ef8feba4b2

- Document database error handling, logging duplication, ban service
timestamp, and orphaned SQLite file issues in Tasks.md
- Bump backend version from 0.9.19-rc.4 to 0.9.19-rc.5

Add granular DB error types with retry logic 9e59fc8bae

New exceptions: DatabaseBusyError, DatabasePermissionDeniedError,
DatabasePathInvalidError, DatabaseCorruptedError, DatabaseUnavailableError.

open_db creates parent directory if missing. Catches all aiosqlite errors
and maps to specific exception types.

get_db retries up to 3x on locked database with backoff.
Propagates specific exceptions instead of generic HTTPException.

Tests for all new error types and retry behavior.

Add logging duplication tests 72273ca945

- test_logging_configuration_no_duplicate_handlers: verify create_app() twice leaves ≤1 StreamHandler
- test_uvicorn_access_logs_go_through_root_handler: verify uvicorn.access can emit JSON via JSONFormatter
- test_external_logging_processor_queues_record: verify _external_logging_processor queues to handler
- test_plain_text_logs_not_emitted_after_startup: verify app.db emits JSON not plain text

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Add tests for since timestamp accuracy in ban_service 407ca83850

- test_since_unix_returns_utc_epoch: validates since_unix('24h') returns UTC epoch
- test_ban_trend_since_is_within_expected_range: validates 23h-ago ban falls in 24h+slack window

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Remove Tasks.md spec, add test for _cleanup_wal_files skipping recent files 408eb900eb

Remove 335-line task specification from Docs/Tasks.md.
Add test confirming _cleanup_wal_files skips recently-modified WAL/SHM files.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

feat: graceful shutdown and WAL cleanup

CI / Backend Tests (pull_request) Has been cancelled

Details

CI / Lint (pull_request) Has been cancelled

Details

CI / Type Check (pull_request) Has been cancelled

Details

CI / Import Boundary (pull_request) Has been cancelled

Details

CI / OpenAPI Breaking Changes (pull_request) Has been cancelled

Details

CI / OpenAPI Baseline Commit (pull_request) Has been cancelled

Details

d13efd4e59

- Add stop_grace_period to backend container for graceful shutdown
- Document WAL mode rationale and orphaned file cleanup in db.py
- Handle database close errors gracefully in lifespan
- Clean up orphaned WAL files during startup before opening DB
- Reorder imports and fix formatting in startup.py

lukas.pupkalipinski merged commit 3af8f0571b into main

2026-05-24 22:05:35 +02:00

lukas.pupkalipinski deleted branch refactoring-backend

2026-05-24 22:05:35 +02:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: lukas.pupkalipinski/BanGUI#4