Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Don't process empty vuln data and remove existing records#46075

Draft
dantecatalfamo wants to merge 5 commits into
mainfrom
45602-vuln-corrupted-download
Draft

Don't process empty vuln data and remove existing records#46075
dantecatalfamo wants to merge 5 commits into
mainfrom
45602-vuln-corrupted-download

Conversation

@dantecatalfamo
Copy link
Copy Markdown
Member

@dantecatalfamo dantecatalfamo commented May 22, 2026

Related issue: Resolves #45602

Checklist for submitter

If some of the following don't apply, delete the relevant line.

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    See Changes files for more information.

Testing

  • Added/updated automated tests
  • QA'd all new/changed functionality manually

@codecov
Copy link
Copy Markdown

codecov Bot commented May 22, 2026

Codecov Report

❌ Patch coverage is 63.38028% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 66.84%. Comparing base (f9c9ad0) to head (24d447e).
⚠️ Report is 35 commits behind head on main.

Files with missing lines Patch % Lines
server/vulnerabilities/oval/sync.go 58.82% 9 Missing and 5 partials ⚠️
...erver/vulnerabilities/goval_dictionary/analyzer.go 14.28% 4 Missing and 2 partials ⚠️
server/vulnerabilities/nvd/cve.go 50.00% 2 Missing and 1 partial ⚠️
server/vulnerabilities/winoffice/analyzer.go 57.14% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #46075      +/-   ##
==========================================
+ Coverage   66.82%   66.84%   +0.01%     
==========================================
  Files        2754     2754              
  Lines      220137   220205      +68     
  Branches    11025    11025              
==========================================
+ Hits       147114   147194      +80     
+ Misses      59727    59694      -33     
- Partials    13296    13317      +21     
Flag Coverage Δ
backend 68.64% <63.38%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

software_cve rows get deleted when vulnerability feed sync fails or gets corrupted

2 participants