Skip to content

Wire audit export and retention env vars into compose - #308

Merged
haksungjang merged 1 commit into
mainfrom
compose-env-wiring/audit-export-retention
Sep 3, 2026
Merged

Wire audit export and retention env vars into compose#308
haksungjang merged 1 commit into
mainfrom
compose-env-wiring/audit-export-retention

Conversation

@haksungjang

Copy link
Copy Markdown
Contributor

Summary

  • 11 vars documented in .env.example (audit-log export webhook batch size/lag/token/URL, audit hash-chain key, retention windows for audit logs / notifications / report downloads / webhook deliveries / Celery task-run records, and the pg_dump/psql subprocess timeout) 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/#307.

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

11 vars documented in .env.example (audit-log export webhook batch
size/lag/token/URL, audit hash-chain key, retention windows for audit
logs / notifications / report downloads / webhook deliveries / Celery
task-run records, and the pg_dump/psql subprocess timeout) 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.

Part of the compose env-var wiring audit.
@haksungjang
haksungjang force-pushed the compose-env-wiring/audit-export-retention branch from 116970a to 6efa8c6 Compare September 3, 2026 07:42
@haksungjang
haksungjang merged commit 1df4e81 into main Sep 3, 2026
24 checks passed
@haksungjang
haksungjang deleted the compose-env-wiring/audit-export-retention branch September 3, 2026 08:03
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