-
Notifications
You must be signed in to change notification settings - Fork 474
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
Unable to suppress the pem file during scan #657
Comments
Any update here team? |
Hi @prathamesh199728, thank you for opening this issue! I'm trying to understand what the use-case would be here. Based on
and
I suppose you'd like detect-secrets to check PEM files in prod, but not in your testing env? Otherwise, could you tell us a bit more about what you're trying to do? I admit I'm a bit confused at the moment. |
For testing purpose some PEM key required which we don't want to highlight during scan. Whitelist this test pem and provide in some file using extension --exclude-files is not feasible solution for us because dev team can pass sensitive material other than PEM also. To avoid this case we need to control those file at command level. Where regex pattern will work for each line of PEM. For example in regex pattern will mention first 2 line of PEM and pass using --exclude-secrets or using --words-list extension where will mention whole file. I tried this at my side but its not working. I am more prefer to go with --words-list extension. |
--words-list this extension only working for single line secret and not for PEM file. |
Have you considered splitting the secrets in two different files and excluding the one used specifically for testing purposes? |
No I guess some confusion here. I have PEM file with below content:
Now I want to ignore above PEM file while scanning so I created one file same as wordlist and put above data in it wordlist.txt
I use this command now:- But above command is not working because logic return in code is scan happened on the basis of each line not for whole content. So need any alternate solution to suppress test pem file content while scanning. |
So you'd like to ignore all data that matches the content of that PEM file? |
correct. |
Sorry, I just would like to make sure I understood this correctly. You have a PEM file containing multiple keys and you'd like to ignore a specific one, thus you created a file containing the key that you ignored and want to pass that file to filter out those keys from the results. Correct? |
Yes this is correct. |
Would the keys you'd like to ignore be present in multiple files, or there'd be just one file containing those keys? |
its multiple files (with same PEM file content) present in different projects and we integrated this tool in CI/CD pipeline so during git push all code get scanned. |
Thank you for the clarification! We'll keep you posted on this issue. |
@lorenzodb1 any update here? |
@prathamesh199728 sorry for the delay in replying. We have accumulated a bit of a backlog and we're trying to get through it. You're more than welcome to open a PR and propose a solution yourself in the meantime :) |
I'm submitting a ...
As the project requirement we need to suppress the pem file for testing purpose. but no option allowed as such.
What is the current behavior?
We dont want to balckilist on file base using paremeters like --exclude-files --exclude-secrets because its hectic and developer can misuse by including sensitive data in this files. Its a compliance issue.
What is the expected behavior?
We want to use the --word-list flag for this requirement but its only working while single string in one line like AWS secrets. When we are trying to use this for PEM key and read it in output its considering every line of PEM key as new one hence it not suppressed.
Need some solutions for such cases.
The text was updated successfully, but these errors were encountered: