You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
and run the soon-to-be-merged keyword detector, then run the audit functionality, it will highlight the first occurrence, and instead of the second. This is a bug.
This is because in audit.py we find the index of the secret
On a related note, w/r/t the Keyword detector, the biggest source of not specific-to-yelp false-positives are variables like FORGOT_PASSWORD: 'forgot_password' where LHS == RHS, maybe I shouldn't flag these, as I've only seen false-positives, but not catching password = 'password' would irk me.
lorenzodb1
added
duplicate
The issue has already already been reported.
bug
The issue describes a malfunctioning aspect of the project.
and removed
pending
The issue still needs to be reviewed by one of the maintainers.
labels
May 9, 2024
If you have e.g.
and run the soon-to-be-merged keyword detector, then run the audit functionality, it will highlight the first occurrence, and instead of the second. This is a bug.
This is because in
audit.py
we find the index of the secretdetect-secrets/detect_secrets/core/audit.py
Line 561 in 1415b4b
and we don't return the index of the secret in the
secret_generator
method of plugins.This somewhat related to the issue of handling multiple secrets on the same line
detect-secrets/detect_secrets/plugins/base.py
Line 76 in 1415b4b
The text was updated successfully, but these errors were encountered: