The Wayback Machine - https://web.archive.org/web/20230218212646/https://github.com/python/cpython/pull/100505
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

[3.11] gh-100474: Fix handling of dirs named index.html in http.server (GH-100475) #100505

Merged
merged 1 commit into from Dec 24, 2022

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Dec 24, 2022

If you had a directory called index.html or index.htm within a directory, it would cause http.server
to return a 404 Not Found error instead of the directory listing. This came about due to not checking
that the index was a regular file.

(cherry picked from commit 46e6a28)

Co-authored-by: James Frost [email protected]

…pythonGH-100475)

If you had a directory called index.html or index.htm within a directory, it would cause http.server to return a 404 Not Found error instead of the directory listing. This came about due to not checking that the index was a regular file.

I have also added a test case for this situation.
(cherry picked from commit 46e6a28)

Co-authored-by: James Frost <[email protected]>
Automerge-Triggered-By: GH:merwok
@bedevere-bot bedevere-bot added type-bug An unexpected behavior, bug, or error awaiting review labels Dec 24, 2022
@merwok merwok added the 🤖 automerge PR will be merged once it's been approved and all CI passed label Dec 24, 2022
merwok
merwok approved these changes Dec 24, 2022
@merwok merwok added 🤖 automerge PR will be merged once it's been approved and all CI passed and removed 🤖 automerge PR will be merged once it's been approved and all CI passed labels Dec 24, 2022
@merwok merwok merged commit 714a93f into python:3.11 Dec 24, 2022
15 checks passed
@miss-islington miss-islington deleted the backport-46e6a28-3.11 branch December 24, 2022 20:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge PR will be merged once it's been approved and all CI passed type-bug An unexpected behavior, bug, or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants