Skip to content

Commit cda821f

Browse files
committed
remove second parameter
1 parent 8379946 commit cda821f

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

packages/core/test/lib/utils/request.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -631,7 +631,7 @@ describe('request utils', () => {
631631
});
632632
});
633633

634-
it('filters comprehensive list of sensitive headers when $description', () => {
634+
it('always filters comprehensive list of sensitive headers', () => {
635635
const headers = {
636636
'Content-Type': 'application/json',
637637
'User-Agent': 'test-agent',

packages/nextjs/src/common/utils/addHeadersAsAttributes.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import type { Span, WebFetchHeaders } from '@sentry/core';
2-
import { getClient, httpHeadersToSpanAttributes, winterCGHeadersToDict } from '@sentry/core';
2+
import { httpHeadersToSpanAttributes, winterCGHeadersToDict } from '@sentry/core';
33

44
/**
55
* Extracts HTTP request headers as span attributes and optionally applies them to a span.
@@ -12,9 +12,6 @@ export function addHeadersAsAttributes(
1212
return {};
1313
}
1414

15-
const client = getClient();
16-
const sendDefaultPii = client?.getOptions().sendDefaultPii ?? false;
17-
1815
const headersDict: Record<string, string | string[] | undefined> =
1916
headers instanceof Headers || (typeof headers === 'object' && 'get' in headers)
2017
? winterCGHeadersToDict(headers as Headers)

0 commit comments

Comments
 (0)