[issue41398] cgi module, parse_multipart fails

[Jeffrey Kintscher]

Change by Jeffrey Kintscher :


--
nosy: +Jeffrey.Kintscher

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41398] cgi module, parse_multipart fails

[Guido van Rossum]

Guido van Rossum  added the comment:

Could you submit a PR then? I don't think I've looked at that module in 20 
years.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41398] cgi module, parse_multipart fails

[Magnus Johnsson]

Magnus Johnsson  added the comment:

No, of course not.
The request is completely valid. Python's cgi library parses it wrong.

The 'resolution' that needs to be done is to fix it in python's source.
That, and the libraries that depend on it, like twisted, probably needs to move 
away from using python's cgi library at all, given the age of this bug.

As it stands, we have had to patch 16 separate calls, and will be moving away 
from the twisted-based server over it anyway, since it seems sketchy.

Going to have a peek at the source, but I am a bit hesitant to touch things 
that have that large a userbase.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41398] cgi module, parse_multipart fails

[Guido van Rossum]

Guido van Rossum  added the comment:

So per the stackoverflow explanation you shouldn’t do that? Should we close 
this?

--
nosy: +gvanrossum

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41398] cgi module, parse_multipart fails

[Magnus Johnsson]

Magnus Johnsson  added the comment:

https://github.com/yohanboniface/falcon-multipart/issues/8

It would seem that the same issue pops up elsewhere. We do indeed set the 
content-type, as per the default examples for unity.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41398] cgi module, parse_multipart fails

[SilentGhost]

Change by SilentGhost :


--
components: +Library (Lib)
nosy: +ethan.furman
type:  -> behavior

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue41398] cgi module, parse_multipart fails

[Magnus Johnsson]

New submission from Magnus Johnsson :

When using the cgi module, parse_multipart fails with the supplied file with 
the error:

Invalid boundary in multipart form: b''

A sample program that demonstrates the error:
import cgi
f = open("60_Request.txt", "r")
print(cgi.parse_multipart(f, {'boundary': 
b'BgTzK0jM20UH01naJdsmAWUj7sqqeoikGZvh3mo9', 'CONTENT-LENGTH': 3992}))

This affects for instance Twisted, and all its dependencies.

--
files: 60_Request.txt
messages: 374307
nosy: Magnus Johnsson
priority: normal
severity: normal
status: open
title: cgi module, parse_multipart fails
versions: Python 3.7, Python 3.8
Added file: https://bugs.python.org/file49340/60_Request.txt

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com