Skip to content

fix: docker nginx 404 error#751

Merged
BilelJegham merged 1 commit intoGeoGuess:masterfrom
HelloJowet:master
May 27, 2024
Merged

fix: docker nginx 404 error#751
BilelJegham merged 1 commit intoGeoGuess:masterfrom
HelloJowet:master

Conversation

@HelloJowet
Copy link
Copy Markdown
Contributor

Types of Changes

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Description

If I run the application with docker and try to access subdirectories like /history, nginx returns a 404 error. I added a nginx configuration file to prevent this issue.

Close #619

How Has This Been Tested?

After my changes I can now access subdirectories like /history directly without any errors.

@netlify
Copy link
Copy Markdown

netlify bot commented May 14, 2024

‼️ Deploy request for geoguess2 rejected.

Name Link
🔨 Latest commit 962b7c2

@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@HelloJowet HelloJowet changed the title fix: docker nginx 404 error fixed fix: docker nginx 404 error May 14, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented May 14, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.12%. Comparing base (4274f7f) to head (962b7c2).
Report is 16 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #751   +/-   ##
=======================================
  Coverage   52.12%   52.12%           
=======================================
  Files          53       53           
  Lines         752      752           
  Branches      209      209           
=======================================
  Hits          392      392           
  Misses        328      328           
  Partials       32       32           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@BilelJegham BilelJegham self-assigned this May 20, 2024
Copy link
Copy Markdown
Member

@BilelJegham BilelJegham left a comment

Choose a reason for hiding this comment

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

Look good, thanks 👍

@BilelJegham
Copy link
Copy Markdown
Member

@all-contributors please add @HelloJowet for code

@allcontributors
Copy link
Copy Markdown
Contributor

@BilelJegham

I've put up a pull request to add @HelloJowet! 🎉

@BilelJegham BilelJegham merged commit a760b65 into GeoGuess:master May 27, 2024
@cypress
Copy link
Copy Markdown

cypress bot commented May 27, 2024

1 failed test on run #713 ↗︎

1 6 0 0 Flakiness 0

Details:

fix: docker nginx 404 error fixed (#751)
Project: GeoGuess Commit: a760b65999
Status: Failed Duration: 01:50 💡
Started: May 27, 2024 11:35 AM Ended: May 27, 2024 11:37 AM
Failed  tests/e2e/specs/game/gameSingleCountry.test.js • 1 failed test • chrome

View Output

Test Artifacts
SinglePlayer > Play SinglePlayer Mode Country Time 10s Test Replay Screenshots

Review all test suite changes for PR #751 ↗︎

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.

[BUG] Docker container returns 404 when visiting relative URLs

2 participants