Skip to content

feat : vuln summary command#738

Merged
bupd merged 4 commits into
goharbor:mainfrom
Sypher845:feat/vuln-summary-command
Mar 15, 2026
Merged

feat : vuln summary command#738
bupd merged 4 commits into
goharbor:mainfrom
Sypher845:feat/vuln-summary-command

Conversation

@Sypher845

@Sypher845 Sypher845 commented Mar 11, 2026

Copy link
Copy Markdown
Contributor

Kooha-2026-03-15-18-42-39

Note : This is the first PR for the issue #723

Description

The PR adds the new harbor vuln summary command. A new command that displays an overview of vulnerability data from Harbor's Security Hub.

  • Severity distribution bar chart (Critical, High, Medium, Low, None, Unknown)
  • Total artifact and scan counts with not-scanned breakdown
  • Optional --artifact flag to show top 5 most vulnerable artifacts
  • Optional ---cve flag to show top 5 most dangerous CVEs

Type of Change

Please select the relevant type.

  • Bug fix
  • New feature
  • Refactor
  • Documentation update
  • Chore / maintenance

@Sypher845 Sypher845 changed the title Feat/vuln summary command Feat : vuln summary command Mar 11, 2026
@codecov

codecov Bot commented Mar 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 2.04082% with 144 lines in your changes missing coverage. Please review.
✅ Project coverage is 7.40%. Comparing base (60ad0bd) to head (823fd81).
⚠️ Report is 114 commits behind head on main.

Files with missing lines Patch % Lines
pkg/views/vulnerability/summary/view.go 0.00% 93 Missing ⚠️
cmd/harbor/root/vulnerability/summary.go 0.00% 29 Missing ⚠️
cmd/harbor/root/vulnerability/cmd.go 0.00% 11 Missing ⚠️
pkg/api/vulnerability_handler.go 0.00% 11 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             main    #738      +/-   ##
=========================================
- Coverage   10.99%   7.40%   -3.59%     
=========================================
  Files         173     265      +92     
  Lines        8671   13168    +4497     
=========================================
+ Hits          953     975      +22     
- Misses       7612   12084    +4472     
- Partials      106     109       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Sypher845 Sypher845 changed the title Feat : vuln summary command feat : vuln summary command Mar 11, 2026
Comment thread pkg/views/vulnerability/summary/view.go Outdated
Comment thread cmd/harbor/root/vulnerability/summary.go Outdated
Comment thread pkg/views/vulnerability/summary/view.go Outdated
Comment thread pkg/views/vulnerability/summary/view.go Outdated

@bupd bupd 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.

@Sypher845 please resolve conflicts

Signed-off-by: Sypher845 <suyashpatil845@gmail.com>
Signed-off-by: Sypher845 <suyashpatil845@gmail.com>
Signed-off-by: Sypher845 <suyashpatil845@gmail.com>
@Sypher845 Sypher845 force-pushed the feat/vuln-summary-command branch from fea5a6e to e596349 Compare March 14, 2026 13:40
@Sypher845

Copy link
Copy Markdown
Contributor Author

@bupd @NucleoFusion done!

@NucleoFusion

Copy link
Copy Markdown
Contributor

@Sypher845 update the description with the new gif/video if its possible

@NucleoFusion NucleoFusion left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

looks good, with some minor QoL changes

Comment thread pkg/views/vulnerability/summary/view.go Outdated
Comment thread pkg/views/vulnerability/summary/view.go Outdated
Comment thread pkg/views/vulnerability/summary/view.go
Signed-off-by: Sypher845 <suyashpatil845@gmail.com>
@Sypher845 Sypher845 force-pushed the feat/vuln-summary-command branch from af68862 to 823fd81 Compare March 15, 2026 15:40

@NucleoFusion NucleoFusion left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm
Thanks for the contribution

@bupd bupd 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.

/lgtm

@bupd bupd merged commit de2bba8 into goharbor:main Mar 15, 2026
6 of 8 checks passed
AMAN-sharma07 pushed a commit to AMAN-sharma07/harbor-cli that referenced this pull request Mar 26, 2026
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.

3 participants