Skip to content

Conversation

@dschom
Copy link
Contributor

@dschom dschom commented Jan 13, 2026

Because

  • We want to upgrade to v10
  • We are adjusting how pii filtering works

This pull request

  • Migrates from Sentry v8 to v10
  • Removes PII filters and other extraneous filters and integrations
  • Adds Sentry.requestDataIntegration() to integration list and removes the deprecated, Sentry.extractRequestData(request.raw.req); approach.
  • Uses app.useGlobalFilters(new SentryGlobalFilter()); for nestjs application which now supports graphql and rest call error handling.
  • Records 'startup' messages on servers to validate sentry message capture is working

Issue that this pull request solves

Closes: FXA-12742

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

image

Other information (Optional)

Any other information that is important to this pull request.

@dschom dschom force-pushed the FXA-12742 branch 3 times, most recently from df70628 to 9737466 Compare January 14, 2026 17:15
@dschom dschom changed the title wip task(many): Upgrade Sentry and reduce filtering Jan 14, 2026
@dschom dschom force-pushed the FXA-12742 branch 3 times, most recently from f4e1c0c to 8b7e677 Compare January 14, 2026 19:26
@dschom dschom marked this pull request as ready for review January 14, 2026 19:43
@dschom dschom requested a review from a team as a code owner January 14, 2026 19:43
@dschom dschom force-pushed the FXA-12742 branch 2 times, most recently from db8b6bd to 05dfd3c Compare January 14, 2026 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants