Uh oh!
There was an error while loading. Please reload this page.
feat(browser): Migrate sendDefaultPii to dataCollection in browser packages - #21097
Conversation
chargome
commented
May 21, 2026
bugbot run |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 2 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit eb71e9e. Configure here.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
size-limit report 📦
|
logaretm
left a comment
There was a problem hiding this comment.
Maybe we should add tests to lock down the new sensitive header filtering behavior
s1gr1d
left a comment
There was a problem hiding this comment.
Looks good, also read the comment from the AI review, but I would consider that a fix (see comment above).
chargome
commented
May 28, 2026
added the tests @logaretm |
Uh oh!
There was an error while loading. Please reload this page.

browser/src/client.ts: Migrate todataCollectionforinfer_iprelay setting andaddAutoIpAddressToSessionlistenerbrowser/src/integrations/httpclient.ts: Split singlesendDefaultPiigate into granularcookies,httpHeaders.request,httpHeaders.responsechecks with new collect behavior filtering. Includes a backwards-compat guard so legacysendDefaultPiiusers see no behavioral change (deferred to v11).browser-utils/src/metrics/utils.ts: Migrate todataCollectionforclient.addressspan attributeNote on backwards compatibility:
When
dataCollectionis not explicitly set by the user, the httpclient integration falls back to the oldsendDefaultPiiboolean gate (all-or-nothing). This avoids silently sending more data for existing users. WhendataCollectionis explicitly set, the granular filtering is applied. (#21094)closes#20929