Skip to content
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

[e2e]: nodejs create main default slsa3 #2298

Closed
ianlewis opened this issue Jun 15, 2023 · 10 comments
Closed

[e2e]: nodejs create main default slsa3 #2298

ianlewis opened this issue Jun 15, 2023 · 10 comments
Labels
area:nodejs Issue related to the Node.js builder e2e e2e integration tests type:bug Something isn't working

Comments

@ianlewis
Copy link
Member

@ianlewis ianlewis added e2e e2e integration tests type:bug Something isn't working area:nodejs Issue related to the Node.js builder labels Jun 15, 2023
@ianlewis ianlewis added this to the Node.js builder GA milestone Jun 19, 2023
ianlewis added a commit that referenced this issue Jun 21, 2023
Fixes getting the filename from the result of `npm pack --json`. Some
older versions return the incorrect file name.

This change falls back and attempts to determine the filename if the
file is not initially found.

Fixes #2290  #2294 #2295 #2296 #2297 #2298 #2299

Signed-off-by: Ian Lewis <[email protected]>
@ianlewis
Copy link
Member Author

enteraga6 pushed a commit to enteraga6/slsa-github-generator that referenced this issue Jul 7, 2023
Fixes getting the filename from the result of `npm pack --json`. Some
older versions return the incorrect file name.

This change falls back and attempts to determine the filename if the
file is not initially found.

Fixes slsa-framework#2290  slsa-framework#2294 slsa-framework#2295 slsa-framework#2296 slsa-framework#2297 slsa-framework#2298 slsa-framework#2299

Signed-off-by: Ian Lewis <[email protected]>
Signed-off-by: Noah Elzner <[email protected]>
@ianlewis ianlewis removed this from the Node.js builder GA milestone Sep 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:nodejs Issue related to the Node.js builder e2e e2e integration tests type:bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant