Skip to content

[3.9] bpo-46436: Fix command-line option -d/--directory in module http.server (GH-30701) #31103

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

Merged
merged 1 commit into from
Feb 14, 2022

Conversation

miss-islington
Copy link
Contributor

@miss-islington miss-islington commented Feb 3, 2022

Fix command-line option -d/--directory in http.server main
function that was ignored when combined with --cgi.

Automerge-Triggered-By: GH:merwok
(cherry picked from commit 2d08034)

Co-authored-by: Géry Ogam [email protected]

https://bugs.python.org/issue46436

Automerge-Triggered-By: GH:merwok

…er (pythonGH-30701)

Fix command-line option -d/--directory in http.server main
function that was ignored when combined with --cgi.

Automerge-Triggered-By: GH:merwok
(cherry picked from commit 2d08034)

Co-authored-by: Géry Ogam <[email protected]>
@miss-islington
Copy link
Contributor Author

@maggyero: Status check is done, and it's a success ✅ .

1 similar comment
@miss-islington
Copy link
Contributor Author

@maggyero: Status check is done, and it's a success ✅ .

@merwok merwok self-assigned this Feb 3, 2022
@miss-islington
Copy link
Contributor Author

@maggyero: Status check is done, and it's a success ✅ .

@merwok
Copy link
Member

merwok commented Feb 8, 2022

@raghavthind2005 Please add a comment with approvals to explain what you did (reviewed the code, built python and ran the tests, reviewed the bug report, etc). Otherwise these drive-by approvals provide zero value and only add fake activity to your profile (and people looking at it can tell that these are not real contributions).

@geryogam
Copy link
Contributor

geryogam commented Feb 8, 2022

Thanks @raghavthind2005 but I concur with @merwok, could you comment about what you did?

@Mariatta Mariatta merged commit 502ad39 into python:3.9 Feb 14, 2022
@miss-islington miss-islington deleted the backport-2d08034-3.9 branch February 14, 2022 20:12
@geryogam
Copy link
Contributor

Thanks @Mariatta!

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.

7 participants