GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-133810: remove `http.server.CGIHTTPRequestHandler` and `--cgi` flag by picnixz · Pull Request #133811 · python/cpython

Conversation

@picnixz

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

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Member
  • Issue: Remove http.server.CGIHTTPRequestHandler and --cgi flag support #133810

📚 Documentation preview 📚: https://cpython-previews--133811.org.readthedocs.build/

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@picnixz
remove http.server.CGIHTTPRequestHandler and --cgi flag
b19ee64
@bedevere-app bedevere-app Bot mentioned this pull request May 10, 2025
Remove http.server.CGIHTTPRequestHandler and --cgi flag support #133810
Closed
@bedevere-app bedevere-app Bot added the awaiting core review label May 10, 2025
picnixz added 2 commits May 10, 2025 11:40
@picnixz
fixup pickle tests
5c5b46c
@picnixz
[trigger ci]
588b38a
@picnixz

picnixz commented May 10, 2025

Copy link
Copy Markdown
Member Author

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@picnixz picnixz mentioned this pull request May 11, 2025
http.client.HTTPMessage.getallmatchingheaders() always returns [] #49303
Closed
picnixz added 2 commits May 11, 2025 09:42
@picnixz
Merge branch 'main' into cleanup/315/remove-cgi-handler-133810
b1c1657
@picnixz
Merge branch 'main' into cleanup/315/remove-cgi-handler-133810
a547ec3
picnixz
picnixz commented May 11, 2025
View reviewed changes
Comment thread Doc/whatsnew/3.15.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
picnixz added 2 commits May 11, 2025 11:18
@picnixz
Update Doc/whatsnew/3.15.rst
4f0d6cd
@picnixz
Merge branch 'main' into cleanup/315/remove-cgi-handler-133810
4175177
@donbarbos donbarbos mentioned this pull request May 14, 2025
gh-131178: Add tests for http.server command-line interface #132540
Merged
@picnixz
Merge branch 'main' into cleanup/315/remove-cgi-handler-133810
80ce094
picnixz
picnixz commented May 16, 2025
View reviewed changes
Comment thread Doc/whatsnew/3.15.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
@picnixz
Update Doc/whatsnew/3.15.rst
2ecff1d
picnixz
picnixz commented May 16, 2025
View reviewed changes
Comment thread Misc/NEWS.d/next/Library/2025-05-10-11-04-47.gh-issue-133810.03WhnK.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
@picnixz
Update Misc/NEWS.d/next/Library/2025-05-10-11-04-47.gh-issue-133810.0…
f9941b0 
…3WhnK.rst
picnixz
picnixz commented May 16, 2025
View reviewed changes
Comment thread Lib/http/client.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
@picnixz
Update Lib/http/client.py
be50cfd
@picnixz
picnixz requested a review from hugovk May 16, 2025 11:01
hugovk
hugovk reviewed May 16, 2025
View reviewed changes
Comment thread Doc/deprecations/pending-removal-in-3.15.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/whatsnew/3.13.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/whatsnew/3.15.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/whatsnew/3.15.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Doc/whatsnew/3.15.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Lib/test/test_httpservers.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Lib/test/test_httpservers.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Lib/test/test_httpservers.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Misc/NEWS.d/next/Library/2025-05-10-11-04-47.gh-issue-133810.03WhnK.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
Comment thread Misc/NEWS.d/next/Library/2025-05-10-11-04-47.gh-issue-133810.03WhnK.rst Resolved

Uh oh!

There was an error while loading. Please reload this page.
picnixz and others added 5 commits May 16, 2025 16:30
@picnixz @hugovk
Apply suggestions from code review
fb781d9 
Co-authored-by: Hugo van Kemenade <[email protected]>
@picnixz
Update test_httpservers.py
782c99d
@picnixz
Update test_httpservers.py
62855fd
@picnixz
Update test_httpservers.py
6bed465
@picnixz @hugovk
Update Misc/NEWS.d/next/Library/2025-05-10-11-04-47.gh-issue-133810.0…
d55959c 
…3WhnK.rst

Co-authored-by: Hugo van Kemenade <[email protected]>
@picnixz
picnixz requested a review from hugovk May 16, 2025 15:28
@hugovk

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.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@picnixz
Merge branch 'main' into cleanup/315/remove-cgi-handler-133810
4938157
picnixz and others added 2 commits May 16, 2025 17:31
@picnixz @hugovk
Update Lib/http/server.py
1978d23 
Co-authored-by: Hugo van Kemenade <[email protected]>
@picnixz
Update server.py
32fb546
hugovk
hugovk approved these changes May 16, 2025
View reviewed changes

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

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

Thanks, nice cleanup! +28 −755

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@bedevere-app bedevere-app Bot added awaiting merge and removed awaiting core review labels May 16, 2025
@picnixz

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 :')
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@hugovk

hugovk commented May 16, 2025

Copy link
Copy Markdown
Member

It's all good! 🚀
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@picnixz picnixz self-assigned this May 16, 2025
@picnixz
picnixz merged commit faac627 into python:main May 17, 2025
@picnixz
picnixz deleted the cleanup/315/remove-cgi-handler-133810 branch May 17, 2025 07:58
@bedevere-app bedevere-app Bot removed the awaiting merge label May 17, 2025
Pranjal095 pushed a commit to Pranjal095/cpython that referenced this pull request Jul 12, 2025
@picnixz @Pranjal095
pythongh-133810: remove http.server.CGIHTTPRequestHandler and `--cg…
450c60b 
…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
@picnixz @taegyunkim
pythongh-133810: remove http.server.CGIHTTPRequestHandler and `--cg…
3fcec37 
…i` flag (python#133811)

The CGI HTTP request handler has been deprecated since Python 3.13.
@methane methane mentioned this pull request Sep 5, 2026
http.server - what is executable on Windows #54692
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hugovk hugovk hugovk approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

@picnixz picnixz

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

2 participants

@picnixz @hugovk
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.