Skip to content

Wire auth/account/permission env vars into compose - #307

Merged
haksungjang merged 1 commit into
mainfrom
compose-env-wiring/auth-account-permissions
Sep 3, 2026
Merged

Wire auth/account/permission env vars into compose#307
haksungjang merged 1 commit into
mainfrom
compose-env-wiring/auth-account-permissions

Conversation

@haksungjang

Copy link
Copy Markdown
Contributor

Summary

  • 9 vars documented in .env.example (self-registration policy, default team role, API-key bookkeeping, permission cache TTL, password-reset and refresh-token retention grace) were never referenced in x-backend-env, so setting them in .env had no effect on the running container.
  • Adds them to both compose files with defaults matching core/config.py exactly.

Context

Part of the same compose env-var wiring audit as #300/#301/#303/#304/#306.

Test plan

  • Both compose files parse as valid YAML
  • Reviewer: this touches the same anchor lines as other PRs in the batch — rebase on merge if needed

9 vars documented in .env.example (self-registration policy, default
team role, API-key bookkeeping, permission cache TTL, password-reset
and refresh-token retention grace) were never referenced in
x-backend-env, so setting them in .env had no effect on the running
container. Adds them with defaults matching core/config.py exactly.

Part of the compose env-var wiring audit.
@haksungjang
haksungjang force-pushed the compose-env-wiring/auth-account-permissions branch from 2728a51 to 09ddd48 Compare September 3, 2026 07:22
@haksungjang
haksungjang merged commit 87c1101 into main Sep 3, 2026
24 checks passed
@haksungjang
haksungjang deleted the compose-env-wiring/auth-account-permissions branch September 3, 2026 07:41
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