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

[Snyk] Security upgrade nginx from 1.25.3-alpine to 1.26.2-alpine #31

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Samsonroyal
Copy link
Collaborator

@Samsonroyal Samsonroyal commented Nov 18, 2024

snyk-top-banner

Snyk has created this PR to fix 5 vulnerabilities in the dockerfile dependencies of this project.

Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.

Snyk changed the following file(s):

  • app/compose/production/nginx/Dockerfile

We recommend upgrading to nginx:1.26.2-alpine, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.

Vulnerabilities that will be fixed with an upgrade:

Issue Score
critical severity Integer Overflow or Wraparound
SNYK-ALPINE318-EXPAT-7908292
  714  
critical severity Integer Overflow or Wraparound
SNYK-ALPINE318-EXPAT-7908293
  714  
high severity CVE-2024-6197
SNYK-ALPINE318-CURL-7569015
  614  
high severity XML External Entity (XXE) Injection
SNYK-ALPINE318-EXPAT-7908294
  614  
high severity Use After Free
SNYK-ALPINE318-LIBXML2-6245694
  614  

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 XML External Entity (XXE) Injection
🦉 Use After Free


Important

Upgrade nginx base image to 1.26.2-alpine in Dockerfile to fix 5 security vulnerabilities.

  • Dockerfile Update:
    • Upgrade nginx base image from 1.25.3-alpine to 1.26.2-alpine in app/compose/production/nginx/Dockerfile.
    • Addresses 5 vulnerabilities including Integer Overflow, XXE Injection, and Use After Free.

This description was created by Ellipsis for 0662ea4. It will automatically update as commits are pushed.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 Looks good to me! Reviewed everything up to 0662ea4 in 7 seconds

More details
  • Looked at 11 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. app/compose/production/nginx/Dockerfile:4
  • Draft comment:
    Consider adding a newline at the end of the file for better readability and to avoid potential issues with some tools.
  • Reason this comment was not posted:
    Confidence changes required: 33%
    The Dockerfile is straightforward and the upgrade is as intended. However, it's a good practice to ensure that the COPY command is followed by a newline for better readability and to avoid potential issues with some tools that might expect a newline at the end of files.

Workflow ID: wflow_zW9DVWJqr4B8dlnd


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

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