mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2024-12-25 12:44:07 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(core): Replace throw with warning when deactivating a non-active workflow (#6969)

Browse source 
Replaces a throw with a warning message instead, since the failure in
question is not serious enough to warrant stopping the application.
This commit is contained in:
Michael Auerswald 2023-08-18 14:04:49 +02:00 committed by GitHub
parent 6422078a5d
commit b6a00febbd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/cli/src/ActiveWorkflowRunner.ts
View file

@ -920,8 +920,10 @@ export class ActiveWorkflowRunner implements IWebhookManager {
// if it's active in memory then it's a trigger
// so remove from list of actives workflows
if (this.activeWorkflows.isActive(workflowId)) {
await this.activeWorkflows.remove(workflowId);
const removalSuccess = await this.activeWorkflows.remove(workflowId);
if (removalSuccess) {
Logger.verbose(`Successfully deactivated workflow "${workflowId}"`, { workflowId });
}
}
}
}

7
packages/core/src/ActiveWorkflows.ts
View file

@ -197,12 +197,13 @@ export class ActiveWorkflows {
*
* @param {string} id The id of the workflow to deactivate
*/
async remove(id: string): Promise<void> {
async remove(id: string): Promise<boolean> {
if (!this.isActive(id)) {
// Workflow is currently not registered
throw new Error(
Logger.warn(
`The workflow with the id "${id}" is currently not active and can so not be removed`,
);
return false;
}
const workflowData = this.workflowData[id];
@ -244,5 +245,7 @@ export class ActiveWorkflows {
}
delete this.workflowData[id];
return true;
}
}