n8n-io/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2026-10-11 22:50:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(core): Simplify webhook request body parsing (no-changelog) (#39031)

Browse Source
This commit is contained in:
Tomi Turtiainen
2026-09-18 07:47:34 +00:00
committed by GitHub
parent 296d1ff6c1
commit 24e9125a7e
1 changed files with 16 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/src/webhooks/webhook-helpers.ts
+16 -14
View File
@@ -1416,6 +1416,18 @@ function requiresMultipartFormData(node: INode): boolean {
);
}
function isParsableContentType(contentType: string | undefined): boolean {
if (!contentType) return false;
return (
contentType.startsWith('application/json') ||
contentType.startsWith('text/plain') ||
contentType.startsWith('application/x-www-form-urlencoded') ||
contentType.endsWith('/xml') ||
contentType.endsWith('+xml')
);
}
/**
* Parses the request body (form, xml, json, form-urlencoded, etc.) if needed
* into the `req.body` property.
@@ -1453,22 +1465,12 @@ async function parseRequestBody(
const { body, cleanup } = await parseFormData(req);
req.body = body;
return cleanup;
} else {
if (nodeVersion > 1) {
if (
contentType?.startsWith('application/json') ||
contentType?.startsWith('text/plain') ||
contentType?.startsWith('application/x-www-form-urlencoded') ||
contentType?.endsWith('/xml') ||
contentType?.endsWith('+xml')
) {
await parseBody(req);
}
} else {
await parseBody(req);
}
}
if (nodeVersion > 1 && !isParsableContentType(contentType)) return undefined;
await parseBody(req);
return undefined;
}