admin/n8n-enterprise-unlocked
1
0
Fork 0
mirror of https://github.com/Abdulazizzn/n8n-enterprise-unlocked.git synced 2025-12-20 11:22:15 +00:00
Code Issues Packages Projects Releases Wiki Activity

refactor(core): Stop reporting to Sentry invalid credentials on workflow activation (no-changelog) (#9555)

Browse Source
This commit is contained in:
Iván Ovejero
2024-05-31 15:41:08 +02:00
committed by GitHub
parent 7fc00d8d10
commit 327794127e
2 changed files with 11 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
packages/workflow/src/errors/workflow-activation.error.ts
View File

@@ -47,6 +47,7 @@ export class WorkflowActivationError extends ExecutionBaseError {
'econnrefused', // Node.js 'econnrefused', // Node.js
'eauth', // OAuth 'eauth', // OAuth
'temporary authentication failure', // IMAP server 'temporary authentication failure', // IMAP server
'invalid credentials',
].some((str) => this.message.toLowerCase().includes(str)) ].some((str) => this.message.toLowerCase().includes(str))
) { ) {
this.level = 'warning'; this.level = 'warning';

13
packages/workflow/test/errors/workflow-activation.error.test.ts
View File

@@ -18,12 +18,15 @@ describe('WorkflowActivationError', () => {
expect(secondError.level).toBe('error'); expect(secondError.level).toBe('error');
}); });
test.each(['ETIMEDOUT', 'ECONNREFUSED', 'EAUTH', 'Temporary authentication failure'])( test.each([
'should set `level` to `warning` for `%s`', 'ETIMEDOUT',
(code) => { 'ECONNREFUSED',
'EAUTH',
'Temporary authentication failure',
'Invalid credentials',
])('should set `level` to `warning` for `%s`', (code) => {
const error = new WorkflowActivationError(code, { cause }); const error = new WorkflowActivationError(code, { cause });
expect(error.level).toBe('warning'); expect(error.level).toBe('warning');
}, });
);
}); });