Skip to content

Enhance release artifact handling andimproved ZIP validation#668

Open
stijnpotters1 wants to merge 2 commits into
masterfrom
fix/truncated-zip-handling
Open

Enhance release artifact handling andimproved ZIP validation#668
stijnpotters1 wants to merge 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

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