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: cve-2023-38545 #981

Merged
merged 1 commit into from
Nov 9, 2023
Merged

fix: cve-2023-38545 #981

merged 1 commit into from
Nov 9, 2023

Conversation

web-mi
Copy link
Contributor

@web-mi web-mi commented Nov 9, 2023

Fixes #

Proposed Changes

Copy link

what-the-diff bot commented Nov 9, 2023

PR Summary

  • Updated Docker Image Reference
    The reference to the Docker image used has been updated in the Dockerfile. This means we changed from using the older version dockerimage.v.001-main to the newer version dockerimage.v.005-main. This will allow us to incorporate the improvements and bug fixes from the newer Docker image, enhancing service reliability and security.

  • Relocated Build Directory
    The build directory, which contains the built, deploy-able version of our application, has been moved to /usr/share/nginx/html. This is a common location for serving web files using nginx, improving the standardization and consistency of our file setup.

  • Configuration File Movement
    The nginx configuration file (nginx.conf) has been relocated to /etc/nginx/conf.d/default.conf. This action aligns with industry-standard locations, improving configuration clarity and maintenance.

Copy link

sonarqubecloud bot commented Nov 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@web-mi web-mi merged commit 4477063 into develop Nov 9, 2023
8 checks passed
@web-mi web-mi deleted the CVE-2023-38545 branch November 9, 2023 11:28
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