-
-
Notifications
You must be signed in to change notification settings - Fork 141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: Handle TAR_ENTRY_INVALID error #1095
Fix: Handle TAR_ENTRY_INVALID error #1095
Conversation
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
|
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
docker.test.js
Outdated
@@ -111,6 +111,20 @@ test("parseImageName tests", () => { | |||
group: "shiftleft", | |||
name: "scan-java", | |||
}); | |||
expect( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I meant docker tests workflow to generate sbom for containers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes added
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
Signed-off-by: Aryan Rajoria <aryanrajoria1003@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you so much!
So far I receive error
Error: TAR_ENTRY_INVALID: checksum failure
instead of theError: TAR_ENTRY_INVALID: linkpath forbidden
.in #572
However, the changes should take into account both of these.
Also Handles:
TAR_ENTRY_ERROR
,TAR_ENTRY_UNSUPPORTED
andTAR_ABORT