mudhorn/n8n
1
0
Fork 0
mirror of https://github.com/n8n-io/n8n.git synced 2025-02-21 02:56:40 -08:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix: Do not crash the server when Telemetry is blocked via DNS (#4947)

Browse source 
* do not crash the process on unhandled axios errors

* postHog.capture does not return a promise
This commit is contained in:
कारतोफ्फेलस्क्रिप्ट™ 2022-12-16 13:23:24 +01:00 committed by GitHub
parent 626879b3a2
commit 6127c958f5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 12 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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

@ -25,11 +25,17 @@ export const initErrorHandling = () => {
release,
environment,
integrations: (integrations) => {
integrations = integrations.filter(({ name }) => name !== 'OnUncaughtException');
integrations.push(new RewriteFrames({ root: process.cwd() }));
return integrations;
},
});
process.on('uncaughtException', (error) => {
ErrorReporterProxy.error(error);
if (error.constructor?.name !== 'AxiosError') throw error;
});
ErrorReporterProxy.init({
report: (error, options) => Sentry.captureException(error, options),
});

9
packages/cli/src/telemetry/index.ts
View file

@ -187,15 +187,12 @@ export class Telemetry {
properties: updatedProperties,
};
if (withPostHog && this.postHog) {
return Promise.all([
this.postHog.capture({
if (withPostHog) {
this.postHog?.capture({
distinctId: payload.userId,
sendFeatureFlags: true,
...payload,
}),
this.rudderStack.track(payload),
]).then(() => resolve());
});
}
return this.rudderStack.track(payload, resolve);