Skip to content

Commit

Permalink
add method to disable defaults
Browse files Browse the repository at this point in the history
  • Loading branch information
mydea committed Jan 25, 2024
1 parent e4fc705 commit af7fc5c
Show file tree
Hide file tree
Showing 4 changed files with 22 additions and 0 deletions.
2 changes: 2 additions & 0 deletions packages/browser/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -60,6 +60,8 @@ export {
browserTracingIntegration,
startBrowserTracingNavigationSpan,
startBrowserTracingPageLoadSpan,
disableDefaultBrowserTracingNavigationSpan,
disableDefaultBrowserTracingPageLoadSpan,
} from '@sentry-internal/tracing';
export type { RequestInstrumentationOptions } from '@sentry-internal/tracing';
export {
Expand Down
16 changes: 16 additions & 0 deletions packages/tracing-internal/src/browser/browserTracingIntegration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -458,6 +458,22 @@ export function startBrowserTracingNavigationSpan(spanOptions: StartSpanOptions)
shouldUseDefaultNavigationSpan = false;
}

/**
* Use this method if you want to disable the default navigation span.
* This is useful if you want to add custom routing instrumentation.
*/
export function disableDefaultBrowserTracingNavigationSpan(disable = true): void {
shouldUseDefaultNavigationSpan = !disable;
}

/**
* Use this method if you want to disable the default page load span.
* This is useful if you want to add custom routing instrumentation.
*/
export function disableDefaultBrowserTracingPageLoadSpan(disable = true): void {
shouldUseDefaultPageLoadSpan = !disable;
}

/** Returns the value of a meta tag */
export function getMetaContent(metaName: string): string | undefined {
// Can't specify generic to `getDomElement` because tracing can be used
Expand Down
2 changes: 2 additions & 0 deletions packages/tracing-internal/src/browser/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ export {
browserTracingIntegration,
startBrowserTracingNavigationSpan,
startBrowserTracingPageLoadSpan,
disableDefaultBrowserTracingNavigationSpan,
disableDefaultBrowserTracingPageLoadSpan,
} from './browserTracingIntegration';

export { instrumentOutgoingRequests, defaultRequestInstrumentationOptions } from './request';
Expand Down
2 changes: 2 additions & 0 deletions packages/tracing-internal/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,8 @@ export {
browserTracingIntegration,
startBrowserTracingNavigationSpan,
startBrowserTracingPageLoadSpan,
disableDefaultBrowserTracingNavigationSpan,
disableDefaultBrowserTracingPageLoadSpan,
BROWSER_TRACING_INTEGRATION_ID,
instrumentOutgoingRequests,
defaultRequestInstrumentationOptions,
Expand Down

0 comments on commit af7fc5c

Please sign in to comment.