mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-01-13 05:47:31 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

fix(core): Fix crash caused by parallel test-webhook calls (#3756)

Browse source 
Fix TestWebhooks undefined properties read
This commit is contained in:
Nathan Poirier 2022-08-10 11:56:23 +02:00 committed by GitHub
parent 7b3d6de44e
commit 8fe71dba4b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
packages/cli/src/TestWebhooks.ts
View file

@ -159,8 +159,10 @@ export class TestWebhooks {
} }
// Remove the webhook // Remove the webhook
clearTimeout(this.testWebhookData[webhookKey].timeout); if (this.testWebhookData[webhookKey]) {
delete this.testWebhookData[webhookKey]; clearTimeout(this.testWebhookData[webhookKey].timeout);
delete this.testWebhookData[webhookKey];
}
// eslint-disable-next-line @typescript-eslint/no-floating-promises // eslint-disable-next-line @typescript-eslint/no-floating-promises
this.activeWebhooks!.removeWorkflow(workflow); this.activeWebhooks!.removeWorkflow(workflow);
}); });