Skip to content

Add word boundaries to Meta tokens

James Liu requested to merge jliu-add-word-boundaries-for-meta-tokens into master

What does this MR do?

!190 (merged) is causing a substantial increase in false positives due to the lax nature of the expressions. This MR adds a leading word boundary \b and trailing boundary ['|\"|\n|\r|\s|\x60] to constrain each pattern. The trailing boundary was taken from existing rules in gitleaks.toml.

This MR itself has an example of a false positive that the revised patterns would ignore:

image

What are the relevant issue numbers?

Does this MR meet the acceptance criteria?

Edited by Lucas Charles

Merge request reports