bug(secret): AWS Secret detection detects strings of 40 characters as false positive #5871
Closed
christiangonre
started this conversation in
False Detection
Replies: 1 comment
-
|
Hello @christiangonre Created #5900 for this task. Regards, Dmitriy |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
IDs
AWS (aws-secret-access-key)
Description
In the version 0.48.0 (I think the problem happens since this fix ) some of my scans fails because it detects some gpg keys as aws-secret-access-key.
My pipelines exits because the aws-secret-access-key is considered as a CRITICAL error that makes sense, but in this case this string is not a secret-access-key.
This is a example of a string that fails:
27C50E7F590947D7273A741E85194C08421980C9This fails in all the trivy scans that involves a secret detection, in my case it fails when I do
trivy imagebut it also fails withtrivy fsReproduction Steps
/file.txt (secrets)
Total: 1 (UNKNOWN: 0, LOW: 0, MEDIUM: 0, HIGH: 0, CRITICAL: 1)
CRITICAL: AWS (aws-secret-access-key)
═══════════════════════════════════════════════════════════════════════════════════════════════════════
AWS Secret Access Key
───────────────────────────────────────────────────────────────────────────────────────────────────────
/file.txt:3
───────────────────────────────────────────────────────────────────────────────────────────────────────
1 ```bash
2 gpg: Signature made Fr 4 Sep 10:04:50 2020 CST
3 [ gpg: using RSA key ****************************************
4 gpg: Good signature from "Sebastian Thiel (YubiKey USB-C) byronimo@gmail.com" [ultimate]
───────────────────────────────────────────────────────────────────────────────────────────────────────
Target
Filesystem
Scanner
Secret
Target OS
Ubuntu 22.04 / MacOS Sonoma 14.2.1
Debug Output
Version
Checklist
-f jsonthat shows data sources and confirmed that the security advisory in data sources was correctBeta Was this translation helpful? Give feedback.
All reactions