Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[Snyk] Security upgrade nginx from 1.25.3-alpine to 1.27.2-alpine #30

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

Conversation

Samsonroyal
Copy link
Collaborator

@Samsonroyal Samsonroyal commented Oct 4, 2024

snyk-top-banner

Snyk has created this PR to fix 4 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.27.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  
critical severity XML External Entity (XXE) Injection
SNYK-ALPINE318-EXPAT-7908294
  714  
high severity CVE-2024-6197
SNYK-ALPINE318-CURL-7569015
  614  
high severity CVE-2024-6197
SNYK-ALPINE318-CURL-7569015
  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


Important

Upgrade nginx base image in Dockerfile to 1.27.2-alpine to address security vulnerabilities.

  • Dockerfile Update:
    • Upgrade nginx base image from 1.25.3-alpine to 1.27.2-alpine in app/compose/production/nginx/Dockerfile to fix security vulnerabilities.
    • No changes to existing RUN and COPY commands.

This description was created by Ellipsis for 6bdfeb8. 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 6bdfeb8 in 6 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:
    Add a newline at the end of the file for POSIX compliance and to avoid potential issues with some tools.
  • Reason this comment was not posted:
    Confidence changes required: 50%
    The Dockerfile lacks a newline at the end of the file, which is a best practice for POSIX compliance and to avoid potential issues with some tools.

Workflow ID: wflow_wojfr2cxfiXT9PM0


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

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

Successfully merging this pull request may close these issues.

2 participants