FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-133810: remove `http.server.CGIHTTPRequestHandler` and `--cgi` flag by picnixz · Pull Request #133811 · python/cpython · GitHub

Repository navigation

gh-133810: remove http.server.CGIHTTPRequestHandler and --cgi flag - #133811

Merged
picnixz merged 19 commits into
python:mainfrom
picnixz:cleanup/315/remove-cgi-handler-133810
May 17, 2025
Merged

picnixz merged 19 commits into
python:mainfrom
picnixz:cleanup/315/remove-cgi-handler-133810

Conversation

picnixz commented May 10, 2025 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

picnixz commented May 10, 2025

Copy link
Copy Markdown
Member Author

Free-threaded failure is known: #132983 (comment).

Comment thread Doc/whatsnew/3.15.rst Outdated
Comment thread Doc/whatsnew/3.15.rst Outdated
Comment thread Lib/http/client.py Outdated
picnixz requested a review from hugovk May 16, 2025 11:01
Comment thread Doc/whatsnew/3.13.rst Outdated
Comment thread Doc/whatsnew/3.15.rst Outdated
Comment thread Doc/whatsnew/3.15.rst Outdated
Comment thread Doc/whatsnew/3.15.rst Outdated
picnixz and others added 5 commits May 16, 2025 16:30
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
…3WhnK.rst

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
picnixz requested a review from hugovk May 16, 2025 15:28

hugovk commented May 16, 2025

Copy link
Copy Markdown
Member

Still some suggestions that GitHub is hiding about not bumping the __version__, and some unused imports in Lib/http/server.py.

picnixz and others added 2 commits May 16, 2025 17:31
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>

hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Thanks, nice cleanup! +28 −755

picnixz commented May 16, 2025

Copy link
Copy Markdown
Member Author

Thanks for the review! Sorry for this long back and forth but I was using GH UI :')

hugovk commented May 16, 2025

Copy link
Copy Markdown
Member

It's all good! 🚀

picnixz self-assigned this May 16, 2025
picnixz merged commit faac627 into python:main May 17, 2025
picnixz deleted the cleanup/315/remove-cgi-handler-133810 branch May 17, 2025 07:58
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
…i` flag (python#133811)

The CGI HTTP request handler has been deprecated since Python 3.13.
taegyunkim pushed a commit to taegyunkim/cpython that referenced this pull request Aug 4, 2025
…i` flag (python#133811)

The CGI HTTP request handler has been deprecated since Python 3.13.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL