refactor(core): Prevent reporting to Sentry IMAP server error (no-cha…

…ngelog) (#9515)
n8n-io · May 27, 2024 · 6ed9ef0 · 6ed9ef0
1 parent 1abb26e
commit 6ed9ef0
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/packages/workflow/src/errors/workflow-activation.error.ts b/packages/workflow/src/errors/workflow-activation.error.ts
@@ -41,7 +41,14 @@ export class WorkflowActivationError extends ExecutionBaseError {
 			return;
 		}
 
-		if (['ETIMEDOUT', 'ECONNREFUSED', 'EAUTH'].some((code) => this.message.includes(code))) {
+		if (
+			[
+				'etimedout', // Node.js
+				'econnrefused', // Node.js
+				'eauth', // OAuth
+				'temporary authentication failure', // IMAP server
+			].some((str) => this.message.toLowerCase().includes(str))
+		) {
 			this.level = 'warning';
 			return;
 		}

diff --git a/packages/workflow/test/errors/workflow-activation.error.test.ts b/packages/workflow/test/errors/workflow-activation.error.test.ts
@@ -18,8 +18,8 @@ describe('WorkflowActivationError', () => {
 		expect(secondError.level).toBe('error');
 	});
 
-	test.each(['ETIMEDOUT', 'ECONNREFUSED', 'EAUTH'])(
-		'should set `level` to `warning` for %s',
+	test.each(['ETIMEDOUT', 'ECONNREFUSED', 'EAUTH', 'Temporary authentication failure'])(
+		'should set `level` to `warning` for `%s`',
 		(code) => {
 			const error = new WorkflowActivationError(code, { cause });