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.24.0 to 1.25.3 #732

Merged
merged 1 commit into from
Feb 15, 2024

Conversation

snyk-io[bot]
Copy link
Contributor

@snyk-io snyk-io bot commented Feb 14, 2024

This PR was automatically created by Snyk using the credentials of a real user.


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

Changes included in this PR

  • Dockerfile

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

Some of the most important vulnerabilities in your base image include:

Severity Priority Score / 1000 Issue Exploit Maturity
critical severity 500 Cleartext Transmission of Sensitive Information
SNYK-DEBIAN11-CURL-3320493
No Known Exploit
critical severity 500 Cleartext Transmission of Sensitive Information
SNYK-DEBIAN11-CURL-3320493
No Known Exploit
high severity 471 Resource Exhaustion
SNYK-DEBIAN11-EXPAT-6227598
No Known Exploit
high severity 400 Improper Verification of Cryptographic Signature
SNYK-DEBIAN11-GNUTLS28-6159417
No Known Exploit
critical severity 500 Integer Overflow or Wraparound
SNYK-DEBIAN11-ZLIB-6008961
No Known Exploit

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

🛠 Adjust project settings


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Resource Exhaustion

@mdelaossa
Copy link
Contributor

@neurosnap while testing Snyk integration we noticed this low hanging fruit to fix ~30 vulnerabilities (you can see the report here)

What do you think? Are we ok to upgrade nginx?

@mdelaossa mdelaossa requested a review from neurosnap February 14, 2024 21:03
Copy link
Member

@neurosnap neurosnap 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!

@neurosnap neurosnap merged commit c0066f8 into main Feb 15, 2024
4 checks passed
@neurosnap neurosnap deleted the snyk-fix-37a4390c30d52926b126a1c85f5af332 branch February 15, 2024 14:16
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