Skip to content

Commit

Permalink
feat(hapi): Warn if hapi is not instrumented
Browse files Browse the repository at this point in the history
  • Loading branch information
mydea committed May 7, 2024
1 parent d64d458 commit e5e130f
Showing 1 changed file with 12 additions and 1 deletion.
13 changes: 12 additions & 1 deletion packages/node/src/integrations/tracing/hapi/index.ts
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import { isWrapped } from '@opentelemetry/core';
import { HapiInstrumentation } from '@opentelemetry/instrumentation-hapi';
import {
SDK_VERSION,
Expand All @@ -11,7 +12,7 @@ import {
} from '@sentry/core';
import { addOpenTelemetryInstrumentation } from '@sentry/opentelemetry';
import type { IntegrationFn } from '@sentry/types';
import { logger } from '@sentry/utils';
import { consoleSandbox, logger } from '@sentry/utils';
import { DEBUG_BUILD } from '../../../debug-build';
import type { Boom, RequestEvent, ResponseObject, Server } from './types';

Expand Down Expand Up @@ -92,4 +93,14 @@ export const hapiErrorPlugin = {
*/
export async function setupHapiErrorHandler(server: Server): Promise<void> {
await server.register(hapiErrorPlugin);

// eslint-disable-next-line @typescript-eslint/unbound-method
if (!isWrapped(server.register)) {
consoleSandbox(() => {
// eslint-disable-next-line no-console
console.warn(
'[Sentry] Hapi is not instrumented. This is likely because you required/imported hapi before calling `Sentry.init()`.',
);
});
}
}

0 comments on commit e5e130f

Please sign in to comment.