fix(node): Skip capturing Hapi Boom responses v8. (#12288)

Recreating #11151 for
v8 again, as the updates seem to be lost when moving the instrumentation
inside the project structure.

The testing updates introduced in the original PR do exist in develop /
master, so this PR only reintroduces changes in the instrumentation
code.

Author: onurtemizkan

packages/node/src/integrations/tracing/hapi/index.ts

@@ -15,7 +15,7 @@ import { logger } from '@sentry/utils';
 import { DEBUG_BUILD } from '../../../debug-build';
 import { generateInstrumentOnce } from '../../../otel/instrument';
 import { ensureIsWrapped } from '../../../utils/ensureIsWrapped';
-import type { Boom, RequestEvent, ResponseObject, Server } from './types';
+import type { RequestEvent, Server } from './types';
 
 const INTEGRATION_NAME = 'Hapi';
 
@@ -38,10 +38,6 @@ const _hapiIntegration = (() => {
  */
 export const hapiIntegration = defineIntegration(_hapiIntegration);
 
-function isBoomObject(response: ResponseObject | Boom): response is Boom {
-  return response && (response as Boom).isBoom !== undefined;
-}
-
 function isErrorEvent(event: RequestEvent): event is RequestEvent {
   return event && (event as RequestEvent).error !== undefined;
 }
@@ -76,9 +72,7 @@ export const hapiErrorPlugin = {
           logger.warn('Isolation scope is still the default isolation scope - skipping setting transactionName');
       }
 
-      if (request.response && isBoomObject(request.response)) {
-        sendErrorToSentry(request.response);
-      } else if (isErrorEvent(event)) {
+      if (isErrorEvent(event)) {
         sendErrorToSentry(event.error);
       }
     });