Skip to content

Fix runtime errors: evict_idle arg, writable temp dir - #3

Merged
mrSamDev merged 1 commit into
mainfrom
fix/runtime-errors
Aug 21, 2026
Merged

Fix runtime errors: evict_idle arg, writable temp dir#3
mrSamDev merged 1 commit into
mainfrom
fix/runtime-errors

Conversation

@mrSamDev

Copy link
Copy Markdown
Owner

Two runtime crashes in the container:

  1. evict_idle() missing 1 required positional argument: idle_seconds. The maintenance loop called the bound method with no args, but it requires the idle threshold. It now passes SESSION_LEASE_TTL_SECONDS.

  2. PermissionError: [Errno 13] Permission denied: temp. When TEMP_DIR is not applied at runtime the app falls back to /app/temp, which is root-owned and not created in the image. The image now pre-creates /app/temp owned by the non-root user, and main() resolves and creates temp/data/backups at startup, failing with a clear message instead of an opaque mid-request crash.

@mrSamDev
mrSamDev merged commit de8445b into main Aug 21, 2026
1 check passed
@mrSamDev
mrSamDev deleted the fix/runtime-errors branch August 21, 2026 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant