Skip to content

fix: make sentry optional#593

Open
hanneshapke wants to merge 2 commits into
mainfrom
fix/make-sentry-optional
Open

fix: make sentry optional#593
hanneshapke wants to merge 2 commits into
mainfrom
fix/make-sentry-optional

Conversation

@hanneshapke

Copy link
Copy Markdown
Collaborator

Description

This PR makes the telemetry reporting opt-in

Linked Issue

Closes #

Testing Done

Still ongoing

Unit tests (Go):

make test-go

Unit tests (Python):

make test

Linting & type checks:

make lint
make typecheck

Manual testing (if applicable):

  • Describe the scenario tested (e.g. PII type detected, proxy request flow, Chrome extension behavior)
  • Include before/after behavior if fixing a bug
  • Note any edge cases verified (e.g. nested PII in JSON payloads, restoration accuracy)

Checklist

  • make test-go and make test pass locally
  • make lint reports no new errors
  • New PII detection changes are covered by unit tests in tests/
  • Go proxy changes include test coverage in src/ or model/
  • Documentation updated if behavior or configuration changed
  • No sensitive data or credentials included

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.

1 participant