Skip to content

fix: enable Kaniko archive (CN-942)#776

Open
ividalATSnyk wants to merge 6 commits intomainfrom
CN-942-fix-kaniko-archive
Open

fix: enable Kaniko archive (CN-942)#776
ividalATSnyk wants to merge 6 commits intomainfrom
CN-942-fix-kaniko-archive

Conversation

@ividalATSnyk
Copy link
Copy Markdown
Contributor

@ividalATSnyk ividalATSnyk commented Apr 2, 2026

  • Ready for review
  • Follows CONTRIBUTING rules
  • Reviewed by Snyk internal team

What does this PR do?

  • enable Kaniko Archive content extraction

Where should the reviewer start?

The change itself is entirely localized to getAndValidateArchivePath(targetImage) in scan.ts

How should this be manually tested?

You can confirm our changes work via:

npx jest test/system/image-type/kaniko-archive.spec.ts
npx jest test/lib/extractor/extractor.spec.ts

Any background context you want to provide?

This error went unnoticed because we weren't actually testing the extractContent flow for archives other than DockerArchive.

What are the relevant tickets?

CN-942

@ividalATSnyk ividalATSnyk force-pushed the CN-942-fix-kaniko-archive branch from 3c10a34 to 58b85bc Compare April 3, 2026 18:41
@ividalATSnyk ividalATSnyk marked this pull request as ready for review April 3, 2026 19:32
@ividalATSnyk ividalATSnyk requested a review from a team as a code owner April 3, 2026 19:32
@ividalATSnyk ividalATSnyk requested a review from adrobuta April 3, 2026 19:32
@snyk-pr-review-bot

This comment has been minimized.

SteveShani
SteveShani previously approved these changes Apr 7, 2026
@snyk-pr-review-bot
Copy link
Copy Markdown

PR Reviewer Guide 🔍

🧪 PR contains tests
🔒 No security concerns identified
⚡ No major issues detected
📚 Repository Context Analyzed

This review considered 5 relevant code sections from 4 files (average relevance: 0.74)

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.

2 participants