David/WFDASH-WEBAPP
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(stock-purchase-send): use DASHBOARD_URL for outcomeUrl (was req.nextUrl.origin)

Browse source 
\`req.nextUrl.origin\` inside the Next.js process resolves to the
container bind address (e.g. \`0.0.0.0:3000\`), which the n8n
container can't reach — the v4 workflow's outcome callback
fails with \`ECONNREFUSED 0.0.0.0:3000\` and the stock PO gets
stuck on \`sending\` because the dashboard never receives the
status-flip callback.

Mirror the customer-PO route: build the callback URL from
\`process.env.DASHBOARD_URL\` (with trailing-slash trim). When
\`DASHBOARD_URL\` is unset the outcomeUrl is empty, and the
workflow's "outcomeUrl missing" branch fires cleanly.

Co-Authored-By: Claude <noreply@anthropic.com>

Co-authored-by: Orca <help@stably.ai>
This commit is contained in:
SJCoetz 2026-08-03 17:44:38 +02:00
parent e795e8584d
commit 6bf8310ef6
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/app/api/v1/stock-purchases/[id]/send-email/route.ts
View file

@ -91,7 +91,12 @@ export async function POST(
const { getAdminPocketBase } = await import("@/lib/server/pocketbase-admin");
const adminPb = await getAdminPocketBase();
const pbUrl = process.env.NEXT_PUBLIC_POCKETBASE_URL || process.env.PB_URL || "";
const dashboardBaseUrl = req.nextUrl.origin;
// Build the callback base URL from DASHBOARD_URL — NOT
// `req.nextUrl.origin`. Inside the Next.js process the origin
// resolves to the bind address (e.g. `0.0.0.0:3000`), which the
// n8n container can't reach. DASHBOARD_URL is the same env var
// the customer-PO route uses for this purpose.
const dashboardBaseUrl = (process.env.DASHBOARD_URL || "").replace(/\/+$/, "");
if (result.failure) {
return NextResponse.json(