Skip to content

Commit

Permalink
"fix" authority regex again
Browse files Browse the repository at this point in the history
  • Loading branch information
Lms24 committed Apr 3, 2023
1 parent 4bb635a commit ea8a521
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 0 additions & 1 deletion packages/sveltekit/src/client/load.ts
Original file line number Diff line number Diff line change
Expand Up @@ -182,7 +182,6 @@ function instrumentSvelteKitFetch(originalFetch: SvelteKitFetch): SvelteKitFetch
const patchedFetchArgs = [input, patchedInit];

if (createSpan) {
U;
fetchPromise = trace(
{
name: `${requestData.method} ${requestData.url}`, // this will become the description of the span
Expand Down
3 changes: 1 addition & 2 deletions packages/utils/src/url.ts
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,7 @@ export function getSanitizedUrlString(url: PartialURL): string {
(host &&
host
// Always filter out authority
// Regex partially taken from: https://stackoverflow.com/a/6165138
.replace(/(?:([^@]*)@)/, '[filtered]:[filtered]@')
.replace(/^.*@/, '[filtered]:[filtered]@')
// Don't show standard :80 (http) and :443 (https) ports to reduce the noise
.replace(':80', '')
.replace(':443', '')) ||
Expand Down

0 comments on commit ea8a521

Please sign in to comment.