From fa394a903d34a3f1c93127b663b8d6e229599baf Mon Sep 17 00:00:00 2001 From: Onur Temizkan Date: Tue, 5 Aug 2025 12:08:03 +0100 Subject: [PATCH] feat(remix): Set sendDefaultPii: true by default --- CHANGELOG.md | 1 + e2e-tests/tests/remix.test.ts | 3 ++- src/remix/sdk-setup.ts | 3 +++ test/remix/client-entry.test.ts | 14 ++++++++++---- 4 files changed, 16 insertions(+), 5 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index a3631305c..58a1521f2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -3,6 +3,7 @@ ## Unreleased - feat(nextjs): Set `sendDefaultPii: true` by default ([#1052](https://github.com/getsentry/sentry-wizard/pull/1052)) +- feat(remix): Set `sendDefaultPii: true` by default ([#1053](https://github.com/getsentry/sentry-wizard/pull/1053)) ## 6.5.0 diff --git a/e2e-tests/tests/remix.test.ts b/e2e-tests/tests/remix.test.ts index d0399db6b..7b96db920 100644 --- a/e2e-tests/tests/remix.test.ts +++ b/e2e-tests/tests/remix.test.ts @@ -195,7 +195,8 @@ function checkRemixProject( })], replaysSessionSampleRate: 0.1, - replaysOnErrorSampleRate: 1 + replaysOnErrorSampleRate: 1, + sendDefaultPii: true })`, ]); }); diff --git a/src/remix/sdk-setup.ts b/src/remix/sdk-setup.ts index 0c05a9128..d9513be6b 100644 --- a/src/remix/sdk-setup.ts +++ b/src/remix/sdk-setup.ts @@ -74,6 +74,7 @@ interface SdkAstOptions { replaysOnErrorSampleRate?: number; integrations?: Array; enableLogs?: boolean; + sendDefaultPii?: boolean; } function getInitCallArgs( @@ -126,6 +127,8 @@ function getInitCallArgs( initCallArgs.replaysSessionSampleRate = 0.1; initCallArgs.replaysOnErrorSampleRate = 1.0; } + + initCallArgs.sendDefaultPii = true; } return initCallArgs; diff --git a/test/remix/client-entry.test.ts b/test/remix/client-entry.test.ts index 37e4177d1..c08065714 100644 --- a/test/remix/client-entry.test.ts +++ b/test/remix/client-entry.test.ts @@ -46,7 +46,8 @@ describe('initializeSentryOnEntryClient', () => { })], replaysSessionSampleRate: 0.1, - replaysOnErrorSampleRate: 1 + replaysOnErrorSampleRate: 1, + sendDefaultPii: true })" `); }); @@ -80,7 +81,8 @@ describe('initializeSentryOnEntryClient', () => { })], replaysSessionSampleRate: 0.1, - replaysOnErrorSampleRate: 1 + replaysOnErrorSampleRate: 1, + sendDefaultPii: true })" `); }); @@ -120,7 +122,9 @@ describe('initializeSentryOnEntryClient', () => { useEffect, useLocation, useMatches - })] + })], + + sendDefaultPii: true })" `); }); @@ -188,7 +192,9 @@ describe('initializeSentryOnEntryClient', () => { useEffect, useLocation, useMatches - })] + })], + + sendDefaultPii: true })" `); });