Skip to content

fix: make sentry optional#593

Merged
hanneshapke merged 4 commits into
mainfrom
fix/make-sentry-optional
Jul 15, 2026
Merged

fix: make sentry optional#593
hanneshapke merged 4 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

@Davidnet Davidnet left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hanneshapke
hanneshapke merged commit 487d526 into main Jul 15, 2026
8 checks passed
@hanneshapke
hanneshapke deleted the fix/make-sentry-optional branch July 15, 2026 19:19
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants