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

Digits regex masks uuids #71

Open
alondahari opened this issue Aug 4, 2023 · 1 comment
Open

Digits regex masks uuids #71

alondahari opened this issue Aug 4, 2023 · 1 comment

Comments

@alondahari
Copy link

The regex for the digits rule matches any number of digits over 4, surrounded by non digits. This rule makes uuid strings become redacted, which is probably unwanted in most logs. Maybe the following regex is slightly better: /(?<!-)\b\d{4,}\b(?!-)/g

@alondahari
Copy link
Author

Furthermore, the rule replaces a possible number with a string ("DIGITS"), which can cause issues as well since it will not be quoted.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant