fix(Respond to Webhook Node): Disable expressions in Respond With (#14727)

This commit is contained in:
Michael Kret
2025-04-18 11:54:28 +03:00
committed by GitHub
parent 38eaef97fe
commit fec2284a77

View File

@@ -6,6 +6,7 @@ import type {
IN8nHttpFullResponse,
IN8nHttpResponse,
INodeExecutionData,
INodeProperties,
INodeType,
INodeTypeDescription,
} from 'n8n-workflow';
@@ -23,39 +24,7 @@ import type { Readable } from 'stream';
import { formatPrivateKey, generatePairedItemData } from '../../utils/utilities';
export class RespondToWebhook implements INodeType {
description: INodeTypeDescription = {
displayName: 'Respond to Webhook',
icon: { light: 'file:webhook.svg', dark: 'file:webhook.dark.svg' },
name: 'respondToWebhook',
group: ['transform'],
version: [1, 1.1],
description: 'Returns data for Webhook',
defaults: {
name: 'Respond to Webhook',
},
inputs: [NodeConnectionTypes.Main],
outputs: [NodeConnectionTypes.Main],
credentials: [
{
name: 'jwtAuth',
required: true,
displayOptions: {
show: {
respondWith: ['jwt'],
},
},
},
],
properties: [
{
displayName:
'Verify that the "Webhook" node\'s "Respond" parameter is set to "Using Respond to Webhook Node". <a href="https://docs.n8n.io/integrations/builtin/core-nodes/n8n-nodes-base.respondtowebhook/" target="_blank">More details',
name: 'generalNotice',
type: 'notice',
default: '',
},
{
const respondWithProperty: INodeProperties = {
displayName: 'Respond With',
name: 'respondWith',
type: 'options',
@@ -103,6 +72,48 @@ export class RespondToWebhook implements INodeType {
],
default: 'firstIncomingItem',
description: 'The data that should be returned',
};
export class RespondToWebhook implements INodeType {
description: INodeTypeDescription = {
displayName: 'Respond to Webhook',
icon: { light: 'file:webhook.svg', dark: 'file:webhook.dark.svg' },
name: 'respondToWebhook',
group: ['transform'],
version: [1, 1.1, 1.2],
description: 'Returns data for Webhook',
defaults: {
name: 'Respond to Webhook',
},
inputs: [NodeConnectionTypes.Main],
outputs: [NodeConnectionTypes.Main],
credentials: [
{
name: 'jwtAuth',
required: true,
displayOptions: {
show: {
respondWith: ['jwt'],
},
},
},
],
properties: [
{
displayName:
'Verify that the "Webhook" node\'s "Respond" parameter is set to "Using Respond to Webhook Node". <a href="https://docs.n8n.io/integrations/builtin/core-nodes/n8n-nodes-base.respondtowebhook/" target="_blank">More details',
name: 'generalNotice',
type: 'notice',
default: '',
},
{
...respondWithProperty,
displayOptions: { show: { '@version': [1, 1.1] } },
},
{
...respondWithProperty,
noDataExpression: true,
displayOptions: { show: { '@version': [{ _cnd: { gte: 1.2 } }] } },
},
{
displayName: 'Credentials',