-
-
Notifications
You must be signed in to change notification settings - Fork 389
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
[Docs] Hint on challenge-1 needs an update (trufflehog) #1193
Comments
It seems trufflehog needs to scan it using generic detector https://github.com/trufflesecurity/trufflehog/blob/main/examples/generic.yml#L15 |
It seems to be version-dependent indeed:
Would you like to file a PR to show how both can be used properly? |
Sure @commjoen I'll prepare a PR. |
Hi @commjoen this command seems not working
I have prepared the PR and it's working with generic detector https://github.com/OWASP/wrongsecrets/pull/1195/files#diff-7cb51d7f736e82d7762d13369216848fc6eeae335a63aab3867542659f6eb825R24 I am wondering, is there a simpler way on trufflehog to scan by default using generic detector? |
Hi @commjoen I am not recommending to use trufflehog version 2.x as it was released > 5 years ago https://github.com/trufflesecurity/trufflehog/tags?after=v3.0.0 What do you think? |
Some installment paths only give version 2 ;-). So I am fine with showing both. |
Haha... ok, because I was wondering how to install the version 2. In that case, I'll just write as it is for version 2 and update the PR. |
so pip3 results in trufflehog 2 :( |
If we can have both that would be awesome! |
Hi @commjoen thanks for the suggestion. PR ready for review now. I have updated the doc. |
Hi there,
I think the hint on challenge-1 needs an update:
The text was updated successfully, but these errors were encountered: