enable selectively listening for a specific webhook

packages/cli/src/webhooks/test-webhooks.ts

@@ -6,6 +6,7 @@ import type {
 	IWorkflowExecuteAdditionalData,
 	IHttpRequestMethods,
 	IRunData,
+	INode,
 } from 'n8n-workflow';
 import { Service } from 'typedi';
 
@@ -225,18 +226,23 @@ export class TestWebhooks implements IWebhookManager {
 		runData?: IRunData,
 		pushRef?: string,
 		destinationNode?: string,
+		webhookNodeName?: string,
 	) {
 		if (!workflowEntity.id) throw new WorkflowMissingIdError(workflowEntity);
 
 		const workflow = this.toWorkflow(workflowEntity);
 
-		const webhooks = WebhookHelpers.getWorkflowWebhooks(
+		let webhooks = WebhookHelpers.getWorkflowWebhooks(
 			workflow,
 			additionalData,
 			destinationNode,
 			true,
 		);
 
+		if (webhookNodeName) {
+			webhooks = webhooks.filter((w) => w.node === webhookNodeName);
+		}
+
 		if (!webhooks.some((w) => w.webhookDescription.restartWebhook !== true)) {
 			return false; // no webhooks found to start a workflow
 		}
@@ -309,6 +315,23 @@ export class TestWebhooks implements IWebhookManager {
 		return true;
 	}
 
+	/**
+	 *
+	 */
+	async isNodeWebhook(
+		node: INode,
+		workflowEntity: IWorkflowDb,
+		additionalData: IWorkflowExecuteAdditionalData,
+	) {
+		if (!workflowEntity.id) throw new WorkflowMissingIdError(workflowEntity);
+
+		const workflow = this.toWorkflow(workflowEntity);
+
+		const webhooks = WebhookHelpers.getWorkflowWebhooks(workflow, additionalData, undefined, true);
+
+		return webhooks.some((w) => w.node === node.name);
+	}
+
 	async cancelWebhook(workflowId: string) {
 		let foundWebhook = false;
 

packages/cli/src/webhooks/webhook-helpers.ts

@@ -87,9 +87,8 @@ export function getWorkflowWebhooks(
 
 			continue;
 		}
-		returnData.push.apply(
+		returnData.push(
-			returnData,
+			...NodeHelpers.getNodeWebhooks(workflow, node, additionalData, ignoreRestartWebhooks),
-			NodeHelpers.getNodeWebhooks(workflow, node, additionalData, ignoreRestartWebhooks),
 		);
 	}
 

packages/cli/src/workflows/workflow-execution.service.ts

@@ -110,7 +110,26 @@ export class WorkflowExecutionService {
 					startNodes?.map((nodeData) => nodeData.name),
 					pinData,
 				);
-		console.log('pinnedTrigger', pinnedTrigger);
+
+		// If pinnedTrigger is a webhook
+		const additionalData = await WorkflowExecuteAdditionalData.getBase(user.id);
+		if (
+			pinnedTrigger &&
+			(await this.testWebhooks.isNodeWebhook(pinnedTrigger, workflowData, additionalData))
+		) {
+			const needsWebhook = await this.testWebhooks.needsWebhook(
+				user.id,
+				workflowData,
+				additionalData,
+				runData,
+				pushRef,
+				destinationNode,
+				pinnedTrigger.name,
+			);
+
+			if (needsWebhook) return { waitingForWebhook: true };
+		}
+
 		// If webhooks nodes exist and are active we have to wait for till we receive a call
 		if (
 			pinnedTrigger === null &&
@@ -119,8 +138,6 @@ export class WorkflowExecutionService {
 				startNodes.length === 0 ||
 				destinationNode === undefined)
 		) {
-			const additionalData = await WorkflowExecuteAdditionalData.getBase(user.id);
-
 			const needsWebhook = await this.testWebhooks.needsWebhook(
 				user.id,
 				workflowData,

packages/cli/src/workflows/workflow.request.ts

@@ -22,6 +22,11 @@ export declare namespace WorkflowRequest {
 		runData: IRunData;
 		startNodes?: StartNodeData[];
 		destinationNode?: string;
+		/**
+		 * Sending a preferredTrigger will override `startNodes`. The execution
+		 * will start from that trigger and it will always be a full manual
+		 * execution.
+		 */
 		preferredTrigger?: string;
 	};
 

packages/workflow/src/Interfaces.ts

@@ -1968,6 +1968,7 @@ export interface INodeHookDescription {
 
 export interface IWebhookData {
 	httpMethod: IHttpRequestMethods;
+	// TODO: rename to nodeName
 	node: string;
 	path: string;
 	webhookDescription: IWebhookDescription;