From af3e2371e63d1f50ff9faedbc870aff63129695c Mon Sep 17 00:00:00 2001 From: Chris Coutinho Date: Sun, 14 Jun 2026 12:59:43 +0200 Subject: [PATCH] docs(env): add required NEXTCLOUD_PUBLIC_ISSUER_URL to login_flow sample MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Round-8 review: env.sample.oauth-multi-user omitted NEXTCLOUD_PUBLIC_ISSUER_URL, which configuration.md marks required for login_flow — a user working from the template alone would hit the "Login URL points to localhost" failure. Add it (with a troubleshooting pointer) and give it + NEXTCLOUD_MCP_SERVER_URL a "REQUIRED: PUBLIC URLs" section header for consistency with the rest of the file. Co-Authored-By: Claude Opus 4.8 (1M context) --- env.sample.oauth-multi-user | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/env.sample.oauth-multi-user b/env.sample.oauth-multi-user index 3ef9962e..f21c4bdb 100644 --- a/env.sample.oauth-multi-user +++ b/env.sample.oauth-multi-user @@ -30,8 +30,13 @@ MCP_DEPLOYMENT_MODE=login_flow NEXTCLOUD_OIDC_CLIENT_ID= NEXTCLOUD_OIDC_CLIENT_SECRET= -# MCP Server URL (for OAuth redirects) +# ===== REQUIRED: PUBLIC URLs ===== +# Public URL of the MCP server (used as the token audience and for OAuth redirects). NEXTCLOUD_MCP_SERVER_URL=http://localhost:8000 +# Public URL of Nextcloud as the user's browser sees it (for Login Flow v2 +# browser redirects). Omitting it causes the "Login URL points to localhost" +# failure — see docs/login-flow-v2.md#troubleshooting. +NEXTCLOUD_PUBLIC_ISSUER_URL=https://nextcloud.example.com # ===== REQUIRED: APP-PASSWORD STORAGE ===== # Required for login_flow — per-user Nextcloud app passwords are stored here