feat(browser): Set page context instead of using request interface

Author: Luca Forstner

dev-packages/browser-integration-tests/suites/integrations/httpContext/test.ts

@@ -4,7 +4,7 @@ import type { Event } from '@sentry/core';
 import { sentryTest } from '../../../utils/fixtures';
 import { getFirstSentryEnvelopeRequest } from '../../../utils/helpers';
 
-sentryTest('httpContextIntegration captures user-agent and referrer', async ({ getLocalTestUrl, page }) => {
+sentryTest('pageInformationIntegration captures user-agent and referrer', async ({ getLocalTestUrl, page }) => {
   const url = await getLocalTestUrl({ testDir: __dirname });
 
   const errorEventPromise = getFirstSentryEnvelopeRequest<Event>(page);

dev-packages/browser-integration-tests/suites/manual-client/browser-context/init.js

@@ -5,7 +5,7 @@ import {
   dedupeIntegration,
   defaultStackParser,
   functionToStringIntegration,
-  httpContextIntegration,
+  pageInformationIntegration,
   eventFiltersIntegration,
   linkedErrorsIntegration,
   makeFetchTransport,
@@ -15,7 +15,7 @@ const integrations = [
   breadcrumbsIntegration(),
   functionToStringIntegration(),
   dedupeIntegration(),
-  httpContextIntegration(),
+  pageInformationIntegration(),
   eventFiltersIntegration(),
   linkedErrorsIntegration(),
 ];

packages/angular/src/sdk.ts

@@ -4,7 +4,7 @@ import {
   breadcrumbsIntegration,
   browserSessionIntegration,
   globalHandlersIntegration,
-  httpContextIntegration,
+  pageInformationIntegration,
   init as browserInit,
   linkedErrorsIntegration,
   setContext,
@@ -41,7 +41,7 @@ export function getDefaultIntegrations(_options: BrowserOptions = {}): Integrati
     globalHandlersIntegration(),
     linkedErrorsIntegration(),
     dedupeIntegration(),
-    httpContextIntegration(),
+    pageInformationIntegration(),
     browserSessionIntegration(),
   ];
 }

packages/browser/src/exports.ts

@@ -88,7 +88,8 @@ export { getDefaultIntegrations, forceLoad, init, onLoad, showReportDialog } fro
 
 export { breadcrumbsIntegration } from './integrations/breadcrumbs';
 export { globalHandlersIntegration } from './integrations/globalhandlers';
-export { httpContextIntegration } from './integrations/httpcontext';
+// eslint-disable-next-line deprecation/deprecation
+export { httpContextIntegration, pageInformationIntegration } from './integrations/pageinformation';
 export { linkedErrorsIntegration } from './integrations/linkederrors';
 export { browserApiErrorsIntegration } from './integrations/browserapierrors';
 

packages/browser/src/integrations/httpcontext.ts

@@ -0,0 +1,41 @@
+import { defineIntegration, getLocationHref } from '@sentry/core';
+import { WINDOW } from '../helpers';
+
+/**
+ * Collects information about the current page and attaches it as context to the event.
+ */
+export const pageInformationIntegration = defineIntegration(() => {
+  return {
+    // TODO(v10): Update name to "PageInformation"
+    name: 'HttpContext',
+    preprocessEvent(event) {
+      // if none of the information we want exists, don't bother
+      if (!WINDOW.navigator && !WINDOW.location && !WINDOW.document) {
+        return;
+      }
+
+      const href = getLocationHref();
+
+      // grab as much info as exists and add it to the event
+      const url = event.request?.url || href;
+      const request = {
+        ...(url && { url }),
+      };
+      event.request = request;
+
+      event.contexts = event.contexts || {};
+      event.contexts.page = {
+        href: href || undefined,
+        referrer: WINDOW.document.referrer,
+        user_agent: WINDOW.navigator.userAgent,
+      };
+    },
+  };
+});
+
+/**
+ * Collects information about the current page and attaches it as context to the event.
+ *
+ * @deprecated This integration was renamed to `pageInformationIntegration`, which should be used instead.
+ */
+export const httpContextIntegration = pageInformationIntegration;

packages/browser/src/integrations/pageinformation.ts

@@ -22,7 +22,7 @@ import { breadcrumbsIntegration } from './integrations/breadcrumbs';
 import { browserApiErrorsIntegration } from './integrations/browserapierrors';
 import { browserSessionIntegration } from './integrations/browsersession';
 import { globalHandlersIntegration } from './integrations/globalhandlers';
-import { httpContextIntegration } from './integrations/httpcontext';
+import { pageInformationIntegration } from './integrations/pageinformation';
 import { linkedErrorsIntegration } from './integrations/linkederrors';
 import { defaultStackParser } from './stack-parsers';
 import { makeFetchTransport } from './transports/fetch';
@@ -43,7 +43,7 @@ export function getDefaultIntegrations(_options: Options): Integration[] {
     globalHandlersIntegration(),
     linkedErrorsIntegration(),
     dedupeIntegration(),
-    httpContextIntegration(),
+    pageInformationIntegration(),
     browserSessionIntegration(),
   ];
 }