[Patches] [ python-Patches-654910 ] Fix bug in IE/CGI [bug 427345]

[SourceForge.net]

Patches item #654910, was opened at 2002-12-16 18:24
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=654910&group_id=5470

Category: Library (Lib)
Group: None
>Status: Open
Resolution: Accepted
Priority: 4
Submitted By: Steve Holden (holdenweb)
Assigned to: Steve Holden (holdenweb)
Summary: Fix bug in IE/CGI [bug 427345]

Initial Comment:
This patch fixes Unix (have_fork) and Windows 
(have_popen2 or have_popen3) CGI scripts to cope with 
Internet Explorer's "mozilla emulation", whereby it 
appears to send a couple extra bytes [CRLF?] at the 
end of requests when KeepAlive isn't in use.

While the bug is believed to only affect requests using 
method POST, the code is written to ignore extra bytes 
in any request triggering a CGI call on these platforms. It 
wasn't obvious how to apply similar fixes to the 
third, "other platform" branch.

The whole *HTTPServer family appears to need some 
sort of unified rewrite, but it's difficult to see how that will 
happen.


----------------------------------------------------------------------

>Comment By: Neal Norwitz (nnorwitz)
Date: 2003-01-08 14:35

Message:
Logged In: YES 
user_id=33168

You need to check out a version on the branch:

    cvs upd -r release22-maint CGIHTTPServer.py
 
Then merge in your last changes:

    cvs upd -j 1.29 -j 1.30 CGIHTTPServer.py

Then you should verify the changes were made correctly:

    cvs diff CGIHTTPServer.py

Then checkin as you would normally:

    cvs commit CGIHTTPServer.py

----------------------------------------------------------------------

Comment By: Steve Holden (holdenweb)
Date: 2003-01-08 14:00

Message:
Logged In: YES 
user_id=88157

Comitted today after review by MvL.

How do I check it into the maintenance branch?

----------------------------------------------------------------------

Comment By: Guido van Rossum (gvanrossum)
Date: 2003-01-08 10:18

Message:
Logged In: YES 
user_id=6380

Looks good to me. Please check this in...

I think it's safe to check this into the 2.2 maintenance
branch as well.

----------------------------------------------------------------------

Comment By: Andrew I MacIntyre (aimacintyre)
Date: 2002-12-17 01:47

Message:
Logged In: YES 
user_id=250749

The patch file is missing.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=654910&group_id=5470