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

__home__ returns 200 instead of 404 #538

Closed
ccchapman opened this issue Jul 17, 2023 · 2 comments
Closed

__home__ returns 200 instead of 404 #538

ccchapman opened this issue Jul 17, 2023 · 2 comments
Labels
bug Something isn't working

Comments

@ccchapman
Copy link

ccchapman commented Jul 17, 2023

Steps to reproduce:

  1. Disable Blitz
  2. Have section with uriFormat: '{slug}'
  3. Create entry in section with slug of __home__
  4. Expect curl -I https://mysite.com to show 200 status code
  5. Expect curl -I https://mysite.com/__home__ to show 404 status code
  6. Enable Blitz
  7. Load https://mysite.com (to warm cache)

Actual behaviour:

curl -I https://mysite.com/__home__ shows 200 status code (returning web/cache/blitz/mysite.com/index.html)

Expected behaviour:

curl -I https://mysite.com/__home__ shows 404 status code

See also craftcms/cms#7585.

@ccchapman ccchapman added the bug Something isn't working label Jul 17, 2023
@bencroker
Copy link
Collaborator

Thanks for reporting this! Fixed in d616f97 for the next release.

@bencroker
Copy link
Collaborator

Released in version 4.5.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants