jpadilla / pyjwt

JSON Web Token implementation in Python
https://pyjwt.readthedocs.io
MIT License
5.18k stars 690 forks source link

Security scan flags up the token being printed #933

Closed CynanX closed 9 months ago

CynanX commented 11 months ago

When including PyJWT in a lambda which I build into a Docker image, I then run Trivy to scan for vulnerabilities and this is picking up a security risk due to a token being exposed.

Expected Result

Trivy passes without exception.

Actual Result

Total: 1 (UNKNOWN: 0, LOW: 0, MEDIUM: 1, HIGH: 0, CRITICAL: 0)
MEDIUM: JWT (jwt-token)
════════════════════════════════════════
JWT token
────────────────────────────────────────
 /var/task/PyJWT-2.8.0.dist-info/METADATA:90 (added by 'COPY /var/task /var/task # buildkit')
────────────────────────────────────────
  88       >>> encoded = jwt.encode({"some": "payload"}, "secret", algorithm="HS256")
  89       >>> print(encoded)
  90 [     *********************************************************************************************************
  91       >>> jwt.decode(encoded, "secret", algorithms=["HS256"])

Reproduction steps

System Information

$ python -m jwt.help
{
  "cryptography": {
    "version": "41.0.2"
  },
  "implementation": {
    "name": "CPython",
    "version": "3.11.6"
  },
  "platform": {
    "release": "23.1.0",
    "system": "Darwin"
  },
  "pyjwt": {
    "version": "2.8.0"
  }
}
github-actions[bot] commented 9 months ago

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days

CynanX commented 9 months ago

Any update on this?

jpadilla commented 9 months ago

I don't particularly think this is something we need to change. I'd suggest looking at how to configure Trivy to mark this as a false positive.

AndreiPaulau commented 9 months ago

Hello,

I agree, that's not pyjwt issue.

But, could you please check the latest comment? https://github.com/aquasecurity/trivy/discussions/5772

Would it be possible to add markers like test or example? {"example": "payload"} or kind of that?

Thanks

deekshi-hari commented 1 month ago

Any solution for this?