fix: pass new optional env vars into the container #13

Merged
jknapp merged 1 commits from fix/pass-new-env-vars into main 2026-07-27 13:14:19 +00:00
Owner

compose environment: is an allow-list — OIDC_ISSUER_MCP never reached the app. Also tolerate empty-string optionals.

🤖 Generated with Claude Code

compose environment: is an allow-list — OIDC_ISSUER_MCP never reached the app. Also tolerate empty-string optionals. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
jknapp added 1 commit 2026-07-27 13:14:17 +00:00
OIDC_ISSUER_MCP and OIDC_AUDIENCE_SCOPE were added to .env and read by the
app, but never reached it: the compose `environment:` block is an explicit
allow-list, not env_file, so anything not named there is silently dropped.
The aud scope only worked because its computed default happened to be right.

Also make optional vars tolerate the empty string. compose renders `${VAR:-}`
as "" rather than omitting the key, so an unset optional var would arrive as
"" and fail .url()/.min(1) validation — taking the app down at boot rather
than falling back to its default.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
jknapp merged commit b35a465303 into main 2026-07-27 13:14:19 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: cybercove-labs/shared-memory#13