Merge next into main: every setting upaas reads in the README, UPAAS_PORT in the Compose file #232

Open
clawbot wants to merge 2 commits from next into main
Collaborator

On next:

  • The README's Configuration table now lists every setting upaas reads, adding UPAAS_MAINTENANCE_MODE, UPAAS_SESSION_SECRET and UPAAS_CORS_ORIGINS, and gives each row its real default and effect (#229).
  • docker-compose.yml now sets UPAAS_PORT to 8080 as well as PORT, so a UPAAS_PORT line in .env no longer moves upaas off the port the compose file's port mapping and healthcheck use (#230).

No code changes.

For deploying, the corrected table rows say:

  • UPAAS_SENTRY_DSN is read but unused; upaas reports nothing to Sentry.
  • UPAAS_DEBUG also sends the session cookie without the Secure flag.
  • Deploys need UPAAS_DATA_DIR to be absolute unless UPAAS_HOST_DATA_DIR is set.
  • UPAAS_PORT is also read and wins over PORT.

Model: opus-5-5

On `next`: - The README's Configuration table now lists every setting upaas reads, adding `UPAAS_MAINTENANCE_MODE`, `UPAAS_SESSION_SECRET` and `UPAAS_CORS_ORIGINS`, and gives each row its real default and effect (https://git.eeqj.de/sneak/upaas/issues/229). - `docker-compose.yml` now sets `UPAAS_PORT` to 8080 as well as `PORT`, so a `UPAAS_PORT` line in `.env` no longer moves upaas off the port the compose file's port mapping and healthcheck use (https://git.eeqj.de/sneak/upaas/issues/230). No code changes. For deploying, the corrected table rows say: - `UPAAS_SENTRY_DSN` is read but unused; upaas reports nothing to Sentry. - `UPAAS_DEBUG` also sends the session cookie without the `Secure` flag. - Deploys need `UPAAS_DATA_DIR` to be absolute unless `UPAAS_HOST_DATA_DIR` is set. - `UPAAS_PORT` is also read and wins over `PORT`. Model: opus-5-5
clawbot added 1 commit 2026-09-29 04:51:02 +02:00
The table left out UPAAS_MAINTENANCE_MODE, UPAAS_SESSION_SECRET and
UPAAS_CORS_ORIGINS, and several rows gave the wrong default or effect.
Each row now matches internal/config/config.go and the code that uses
the value: UPAAS_PORT is also read and wins over PORT, UPAAS_DATA_DIR
must be absolute for deploys unless UPAAS_HOST_DATA_DIR is set,
UPAAS_HOST_DATA_DIR falls back to UPAAS_DATA_DIR and must be absolute
when set, UPAAS_DEBUG also drops the session cookie's Secure flag,
UPAAS_SENTRY_DSN is not used, and /metrics exists only when
UPAAS_METRICS_USERNAME is set. A sentence under the table names the
standard Docker client variables. TODO.md records the step.

Model: opus-5-5
Co-authored-by: clawbot <sneak+clawbot@sneak.cloud>
clawbot added the merge-ready label 2026-09-29 04:51:07 +02:00
sneak was assigned by clawbot 2026-09-29 04:51:08 +02:00
clawbot added 1 commit 2026-09-29 05:26:54 +02:00
Set UPAAS_PORT as well as PORT in docker-compose.yml (closes #230)
Check / check (pull_request) Successful in 5m13s
0a94484795
upaas reads UPAAS_PORT before PORT, so a UPAAS_PORT line in .env made it
listen on another port than the one the compose file's port mapping and
healthcheck use, and the container never became healthy. The compose file
now sets both to 8080, and its comment and the README's Docker Compose
section say that both are set.

Model: opus-5-5
clawbot changed title from Merge next into main: every setting upaas reads in the README to Merge next into main: every setting upaas reads in the README, UPAAS_PORT in the Compose file 2026-09-29 05:27:01 +02:00
All checks were successful
Check / check (pull_request) Successful in 5m13s
Required
Details
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin next:next
git checkout next
Sign in to join this conversation.