fix: rename onuncaughtexception error handler internal tag (#161)

* fix: rename onuncaughtexception error handler internal tag * docs: add changeset
hyperdxio · Jun 25, 2024 · f7183db · f7183db
1 parent d332c32
commit f7183db
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/.changeset/loud-paws-film.md b/.changeset/loud-paws-film.md
@@ -0,0 +1,6 @@
+---
+'@hyperdx/instrumentation-exception': patch
+'@hyperdx/node-opentelemetry': patch
+---
+
+fix: rename onuncaughtexception error handler internal tag
diff --git a/packages/instrumentation-exception/src/node/integrations/onuncaughtexception.ts b/packages/instrumentation-exception/src/node/integrations/onuncaughtexception.ts
@@ -58,7 +58,7 @@ export const onUncaughtExceptionIntegration = defineIntegration(
   },
 );
 
-type ErrorHandler = { _errorHandler: boolean } & ((error: Error) => void);
+type ErrorHandler = { _hdxErrorHandler: boolean } & ((error: Error) => void);
 
 /** Exported only for tests */
 export function makeErrorHandler(
@@ -87,7 +87,7 @@ export function makeErrorHandler(
           // as soon as we're using domains this listener is attached by node itself
           listener.name !== 'domainUncaughtExceptionClear' &&
           // the handler we register in this integration
-          (listener as ErrorHandler)._errorHandler !== true
+          (listener as ErrorHandler)._hdxErrorHandler !== true
         );
       }).length;
 
@@ -156,6 +156,6 @@ export function makeErrorHandler(
         }
       }
     },
-    { _errorHandler: true },
+    { _hdxErrorHandler: true },
   );
 }