Skip to content

Fix: keep settings saves out of nginx temporary files - #471

Merged
SandObserver merged 1 commit into
mainfrom
fix/nginx-api-body-buffer
Oct 17, 2026
Merged

SandObserver merged 1 commit into
mainfrom
fix/nginx-api-body-buffer

Conversation

@SandObserver

Copy link
Copy Markdown
Owner

Summary

  • Set client_body_buffer_size 3m on the /api/ location, matching client_max_body_size.
  • A config save over nginx's 16 KB default was written to /var/lib/nginx/tmp/client_body/ and logged a warning on every save. The wallpaper upload location keeps the default.
  • Add a test that the /api/ buffer is at least the API's own body limit.

Testing
All green, e2e included. Added 1 nginx config test.

Changelog

Fixed

  • Stop logging an nginx warning on every settings save.

Checklist

  • Stays within the project constraints: one container, no runtime dependencies, no build step (see CONTRIBUTING.md).
  • Tests added or updated for new logic, and the full suite, lint, and typechecks pass.
  • Docs updated if behavior or configuration changed.

@SandObserver SandObserver added the docker Pull requests that update docker code label Oct 17, 2026
Set client_body_buffer_size 3m on the /api/ location. A saved config
over 16 KB was written to a temporary file with a warning on every save.
@SandObserver
SandObserver force-pushed the fix/nginx-api-body-buffer branch from 4635c8c to 4c905e6 Compare October 17, 2026 14:21
@SandObserver
SandObserver merged commit cc31563 into main Oct 17, 2026
6 checks passed
@SandObserver
SandObserver deleted the fix/nginx-api-body-buffer branch October 17, 2026 14:27
stackyard-release Bot pushed a commit that referenced this pull request Oct 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docker Pull requests that update docker code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant