Skip to content

security: suppress GO-2026-5932 (openpgp not used in this codebase) - #51

Open
sudeeshjohn wants to merge 2 commits into
IBM:mainfrom
sudeeshjohn:main
Open

sudeeshjohn wants to merge 2 commits into
IBM:mainfrom
sudeeshjohn:main

Conversation

@sudeeshjohn

Copy link
Copy Markdown
Collaborator

The Trivy scan flags golang.org/x/crypto for GO-2026-5932 which covers the golang.org/x/crypto/openpgp sub-packages (unmaintained, unsafe by design, no fixed version available).

golang.org/x/crypto is an indirect dependency pulled in via github.com/IBM/infra-go-sdk for its ssh sub-package only — openpgp is never imported anywhere in this codebase — so the vulnerability is not reachable.

Adding .trivyignore to suppress the false-positive advisory closes code scanning alert #16.

Ref: https://pkg.go.dev/vuln/GO-2026-5932

Signed-off-by: SUDEESH JOHN sudeeshjohn@in.ibm.com

@sudeeshjohn
sudeeshjohn force-pushed the main branch 2 times, most recently from b17fa2c to a2ea53a Compare September 19, 2026 01:25
Signed-off-by: SUDEESH JOHN <sudeeshjohn@in.ibm.com>
The Trivy scan flags golang.org/x/crypto for GO-2026-5932 which covers
the golang.org/x/crypto/openpgp sub-packages (unmaintained, unsafe by
design, no fixed version available).

golang.org/x/crypto is an indirect dependency pulled in via
github.com/IBM/infra-go-sdk for its ssh sub-package only — openpgp is
never imported anywhere in this codebase — so the vulnerability is not
reachable.

Add .trivyignore to suppress the false-positive advisory.

Ref: https://pkg.go.dev/vuln/GO-2026-5932
Signed-off-by: SUDEESH JOHN <sudeeshjohn@in.ibm.com>

@Karthik-K-N Karthik-K-N left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see from the doc that the advise to use github.com/ProtonMail/go-crypto/openpgp but given its indirect dependency and we are not using it should be good.

/lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants