Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

Updated Data Scrubbing #44

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

matthew-gerstman
Copy link

⚠️ Need to update documentation to reflect changes.

Issue #9

Updated the data scrubbing code to allow for users to include their own custom processors. It also validates against a few more "bad keys" such as "authorization", "apikey", and "api_key." I took these keys from raven-python

I also added value validation to scrub out anything that looks like a credit card number.
Also it's now scrubbing HTTP Headers.

This is a more thorough implementation of: http://sentry.readthedocs.org/en/latest/developer/client/#scrubbing-data

Note: This is an API breaking change as NewClient now takes a ClientConfig struct to handle both tags/processors

Edit:
I wrote a processor_tests.go file that includes a test of the default processor and a custom provided one.

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.

None yet

1 participant