Compare commits

...
Author SHA1 Message Date
shadowdaoandClaude Opus 5 1a44362202 fix: pass OIDC_OFFLINE_ACCESS through to the app container
The app service's environment block is an explicit allow-list, not
env_file — as the comment two lines above this change says, a var added
to .env but not listed here never reaches the container.

PR #20 added OIDC_OFFLINE_ACCESS to .env.example and documented it in the
README but missed this, so setting it in .env would have been silently
ignored and the refresh-token fix would not have taken effect. Caught
while deploying, before it could look like the IdP change had failed.

The external-db override merges its environment block rather than
replacing it, so it inherits this and needs no equivalent change.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-08-11 15:29:05 -07:00
jknapp e0d0d3b2b3 Merge pull request 'fix: let deployments advertise offline_access so MCP sessions can refresh' (#20) from fix/oauth-offline-access into main 2026-08-11 22:21:51 +00:00
+1
View File
@@ -117,6 +117,7 @@ services:
# added to .env but not listed here never reaches the container.
OIDC_ISSUER_MCP: ${OIDC_ISSUER_MCP:-}
OIDC_AUDIENCE_SCOPE: ${OIDC_AUDIENCE_SCOPE:-}
OIDC_OFFLINE_ACCESS: ${OIDC_OFFLINE_ACCESS:-}
PLUGIN_MARKETPLACE_URL: ${PLUGIN_MARKETPLACE_URL:-}
PLUGIN_MARKETPLACE_NAME: ${PLUGIN_MARKETPLACE_NAME:-shared-memory}