Skip to content
This repository was archived by the owner on Oct 23, 2023. It is now read-only.

2.0.x Problem: adding additional tags to be sanitised - #1250

Open
savin-berlin wants to merge 3 commits into
masterfrom
2.0.x
Open

2.0.x Problem: adding additional tags to be sanitised#1250
savin-berlin wants to merge 3 commits into
masterfrom
2.0.x

Conversation

@savin-berlin

@savin-berlinsavin-berlin commented May 29, 2018

Copy link
Copy Markdown

I have a Problem with raven.processors.SanitizePasswordsProcessor while adding additional items which need to be sanitized.
If I add additional items to be sanitized, it doesn't works for additional tags which I send with the merge function -> client.context.merge({'tags': "t"})

These is two ways how I tried to add additional sanitize_items

  1. sanitize_keys = ['_consumer_key', '_consumer_secret', '_access_token', '_access_token_secret'],
    or
  2. client.module_cache['raven.processors.SanitizePasswordsProcessor'].KEYS = frozenset(['sentry_dsn', 'password', 'passwd', 'access_token', 'secret', 'apikey', 'api_key', 'authorization', '_consumer_key', '_consumer_secret', '_access_token', '_access_token_secret'])

And unfortunately those keys are still be send to the server in the bunch of tags.

U would really happy about your responce=)

Sunny Week
Egor

glasslionand others added 3 commits November 18, 2014 14:56
This bug was fixed in Raven 3.1.1, but the patch was never backported to
raven 2.x. This commit simply patch the commit 4431729
to the raven 2.0.12.2 code.
@savin-berlinsavin-berlin changed the title 2.0.x add tags to be sanitized also2.0.x Problem: adding additional tags to be sanitisedMay 29, 2018
Sign up for freeto 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.

3 participants

@savin-berlin@glasslion@xordoquy