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

Fix/dockerfile instructions #288

Merged
merged 2 commits into from
Nov 2, 2020
Merged

Conversation

ivanstanev
Copy link
Contributor

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

What does this PR do?

This fixes a regression with rolling out static scanning. Previously dependencies used to be annotated with the Dockerfile instruction that introduced them. Without this fix we could not properly distinguish between packages installed by the base OS and packages installed by user instructions in the Dockerfile. We were incorrectly representing all packages as coming from the base image.

What are the relevant tickets?

Jira ticket RUN-1263

This fixes a regression with rolling out static scanning. Previously dependencies used to be annotated with the Dockerfile instruction that introduced them. Without this fix we could not properly distinguish between packages installed by the base OS and packages installed by user instructions in the Dockerfile. We were incorrectly representing all packages as coming from the base image.
@ivanstanev ivanstanev requested a review from a team as a code owner November 2, 2020 17:07
@ivanstanev ivanstanev self-assigned this Nov 2, 2020
@github-actions
Copy link

github-actions bot commented Nov 2, 2020

Expected release notes (by @ivanstanev)

fixes:
annotate packages with the instruction that introduced them (b04baa6)

others (will not be included in Semantic-Release notes):
stop Jest on first test failure (af4794f)

  • I hereby acknowledge these release notes are 🥙 AWESOME 🥙

@ivanstanev ivanstanev merged commit e789fa1 into master Nov 2, 2020
@ivanstanev ivanstanev deleted the fix/dockerfile-instructions branch November 2, 2020 17:18
@snyksec
Copy link

snyksec commented Nov 2, 2020

🎉 This PR is included in version 4.6.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants