Skip to content

feat(wupnext): add new env vars for mcp server - #1100

Open
javierlnmn wants to merge 1 commit into
Dokploy:canaryfrom
javierlnmn:update-wupnext-template-mcp
Open

feat(wupnext): add new env vars for mcp server#1100
javierlnmn wants to merge 1 commit into
Dokploy:canaryfrom
javierlnmn:update-wupnext-template-mcp

Conversation

@javierlnmn

Copy link
Copy Markdown
Contributor

What is this PR about?

Adds new env vars for MCP server + Django Q config missing ones.

Checklist

Before submitting this PR, please make sure that:

Issues related (if applicable)

Close automatically the related issues using the keywords: closes #ISSUE_NUMBER

Screenshots or Videos

image

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 21, 2026
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown
built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
templates ✅ Ready (View Log) Visit Preview 307b557

@Siumauricio

Copy link
Copy Markdown
Contributor

Reviewed and deployed the PR version of blueprints/wupnext on a Dokploy instance.

Upstream check (javierlnmn/wupnext, cloned at 6eeb579): all five new vars exist in .env.example and wupnext/settings.py:

  • SITE_URL (settings.py:218, added 2026-07-27) — used for links in emails, password reset raises if unset.
  • MCP_BASE_URL (settings.py:285, added 2026-08-20 in "Create base MCP server with OAuth and DCR") — feeds OAUTH2_PROTECTED_RESOURCE_IDENTIFIER / authorization servers.
  • Q_WORKERS / Q_TIMEOUT / Q_RETRY (settings.py:224-226) — Django Q2 cluster config, defaults match the template (2 / 60 / 120).

Deploy (PR version):

  • generate-meta.js --check, validate-docker-compose.ts, validate-template.ts all pass.
  • Both containers running, RestartCount: 0; env injected as expected (SITE_URL/MCP_BASE_URL on web, SITE_URL/Q_* on worker).
  • Worker log: Q Cluster ... running, 2 worker processes ready. Web log: migrations applied, superuser created, gunicorn listening on :8000.
  • GET / → 302 → /accounts/login/ → 200 «WupNext» through Traefik.

One thing to fix before merging: the image the template pins (docker.io/javierlnmn/wupnext:latest) was last pushed on 2026-08-17 (same digest as 0.2.0), but the MCP server landed upstream on 2026-08-20. On the deployed instance GET /mcp, /.well-known/oauth-protected-resource and /.well-known/oauth-authorization-server all return 404, so the new description ("...and an MCP server so an AI client can work the board") and the mcp tag aren't true for the image users will actually pull today. MCP_BASE_URL is simply ignored by that build.

Could you push a new image that includes the MCP server (e.g. 0.3.0 + latest)? Ideally also pin the compose to that tag instead of :latest so the template is reproducible — I'm happy to re-test once it's up. Everything else in the PR is good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants