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

Scrub fields should be a suffix match #160

Open
rokob opened this issue Mar 31, 2017 · 2 comments
Open

Scrub fields should be a suffix match #160

rokob opened this issue Mar 31, 2017 · 2 comments

Comments

@rokob
Copy link
Contributor

rokob commented Mar 31, 2017

See the discussion on #149 for how the scrubbing code can be improved. This issue is so this idea doesn't get lost in the mix.

@rivkahstandig3636
Copy link
Contributor

Hi there, I’m closing out all issues opened before 2018 that haven’t had any activity on them since the start of this year. If this is still an issue for you, please comment here and we can reopen this. Thanks!

@AlecRosenbaum
Copy link

Can we open this issue back up?

I know it's old, but the contents of #149 are still relevant today. Particularly the great comment here: #149 (comment), which also lays out some clear nexts steps pending the maintainer's feedback.

The callable support seems relatively easy and flexible, though the use case I'm after is having something like a _secret suffix. Then I don't have to add all my secret variable names to the scrub list. Generally, overscrubbing is preferable to leaking secret data IMO.

@brianr brianr reopened this Sep 28, 2020
@bxsx bxsx removed this from the v0.15.0 milestone Jan 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants