Skip to content

Add support for sonatype nancy vulnerability scanning#153

Merged
hantmac merged 4 commits intoopenkruise:masterfrom
Parship999:sonatype-nancy-vulnerability-scanning-new
Jul 22, 2025
Merged

Add support for sonatype nancy vulnerability scanning#153
hantmac merged 4 commits intoopenkruise:masterfrom
Parship999:sonatype-nancy-vulnerability-scanning-new

Conversation

@Parship999
Copy link
Copy Markdown

Which Issue this PR solving?

Changes Made

  • Files Modified
    • .github/workflows/ci.yaml
    • .nancy-ignore (New file)
    • README.md

Security Tools Implemented

  • Nancy by sonatype
    • Scans all Go dependencie against sonatype OSS Index
    • comprehensive vulnerability database coverage
    • provides detail CVE information and CVSS scores
  • govulncheck (official Go tool)
    • uses call graph analysis to reduce false positives
    • only reports vulnerabilities in actually-used code paths

How It Works

  • Runs on every push to master and release branches
  • Runs on every pull request
  • generates dependency list using go list -json -deps
  • scans dependencies with both tools
  • fails build if vulnerabilities are found

Testing Results

  • Tested locally on current codebase
Screenshot 2025-07-14 223502

Why use Nancy + govulncheck Together?

  • think of it like this:
    • Nancy checks everything , it looks through all your project’s dependencies and flags known vulnerabilities, even the ones you're not using directly.
    • govulncheck goes deeper, it checks your actual code to see if you’re really using anything dangerous.

Together, they arre the perfect combo

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
Parship Chowdhury added 2 commits July 15, 2025 15:58
Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
@Parship999
Copy link
Copy Markdown
Author

@furykerry @FillZpp @hantmac I have fixed the vulnerability scan check. To pass it i have updated the go version, that's why the golangci lint is failing.

@kruise-bot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: hantmac

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@hantmac hantmac merged commit 7c240e5 into openkruise:master Jul 22, 2025
3 of 5 checks passed
@Parship999
Copy link
Copy Markdown
Author

Thanks

GautamBytes pushed a commit to GautamBytes/kruise-tools that referenced this pull request Aug 20, 2025
* added sonatype nancy vulnerability scanning

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>

* fix 1

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>

* fix 2

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>

* vulnerability check fixed

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>

---------

Signed-off-by: Parship Chowdhury <i.am.parship@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat/Security] Add support for sonatype-nexus-community/nancy

3 participants