Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add ignoreList to redaction processor #18680

Closed
mterhar opened this issue Feb 15, 2023 · 1 comment · Fixed by #18681
Closed

Add ignoreList to redaction processor #18680

mterhar opened this issue Feb 15, 2023 · 1 comment · Fixed by #18681
Labels
enhancement New feature or request needs triage New item requiring triage processor/redaction Redaction processor

Comments

@mterhar
Copy link
Member

mterhar commented Feb 15, 2023

Component(s)

processor/redaction

Is your feature request related to a problem? Please describe.

Some fields may have a 12 digit number but it's an internal system identifier which doesn't benefit from redaction.
It would be nice if these attributes could be passed through without having the value treated as a credit card number.

Describe the solution you'd like

Add an ignorelist to the redaction processor so it can be explicitly set to pass suspicious data.

Describe alternatives you've considered

Other processors don't seem to have the regex matching and replacement ability that the redaction processor has.

Additional context

The "allowList" in the processor seems to provide this capabillity at first glance but it is actually different. Adding the ignoreList will give users a way to proceed without having the modify their system identifiers or other drastic measures.

@mterhar mterhar added enhancement New feature or request needs triage New item requiring triage labels Feb 15, 2023
@github-actions github-actions bot added the processor/redaction Redaction processor label Feb 15, 2023
@github-actions
Copy link
Contributor

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs triage New item requiring triage processor/redaction Redaction processor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant