Skip to content

Enhance release artifact handling andimproved ZIP validation - #668

Merged
stijnpotters1 merged 2 commits into
masterfrom
fix/truncated-zip-handling
Jul 27, 2026
Merged

Enhance release artifact handling andimproved ZIP validation#668
stijnpotters1 merged 2 commits into
masterfrom
fix/truncated-zip-handling

Conversation

@stijnpotters1

Copy link
Copy Markdown
Contributor

The download of this release was probabily truncated by a pod restart. So it could not scan this release since it was corrupt
Now when it sees its not a valid and complete zip so it is removed and redownloaded.

Now with this new implementation this 8.0.5 release is being redownloaded and scanned right:

2026-07-21T13:41:37.853+02:00  INFO 38360 --- [insights] [           main] o.f.i.v.VulnerabilityService             : Java Index DB succesfully updated.
2026-07-21T13:41:37.855+02:00  INFO 38360 --- [insights] [           main] o.f.i.v.VulnerabilityService             : Start processing release: v8.0.5
2026-07-21T13:41:38.022+02:00  WARN 38360 --- [insights] [           main] o.f.i.release.ReleaseArtifactService     : Cached ZIP for v8.0.5 is corrupt (likely a truncated download), re-downloading: \release-archive\v8.0.5.zip
2026-07-21T13:41:38.044+02:00  INFO 38360 --- [insights] [           main] o.f.i.release.ReleaseArtifactService     : ZIP not found op storage, downloading: v8.0.5
2026-07-21T13:41:49.409+02:00  INFO 38360 --- [insights] [           main] o.f.i.release.ReleaseArtifactService     : ZIP succesfully downloading to storage: \release-archive\v8.0.5.zip

@stijnpotters1 stijnpotters1 self-assigned this Jul 21, 2026
@stijnpotters1 stijnpotters1 linked an issue Jul 21, 2026 that may be closed by this pull request
@stijnpotters1
stijnpotters1 requested a review from Matthbo July 21, 2026 11:49

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

Sonar warning, e instead of exception, unused variables

@sonarqubecloud

Copy link
Copy Markdown

@stijnpotters1
stijnpotters1 merged commit 3b1af64 into master Jul 27, 2026
5 checks passed
@stijnpotters1
stijnpotters1 deleted the fix/truncated-zip-handling branch July 27, 2026 13:13
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.

Zip END header error during cve scan

2 participants