[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-16 Thread Ian Bicking
Ian Bicking <[EMAIL PROTECTED]> added the comment: cgi started using this argument due to the potential of a DoS attack without the length limit. So undoing this in cgi (even as an option) would be a regression. ___ Python tracker <[EMAIL PROTECTED]>

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-16 Thread Phillip J. Eby
Phillip J. Eby <[EMAIL PROTECTED]> added the comment: Then obviously it makes no sense to update wsgiref before the spec. ISTM the correct way to deal with this is update the cgi module to include a WSGI-compatible API. ___ Python tracker <[EMAIL PROTECTED]

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Ian Bicking
Ian Bicking <[EMAIL PROTECTED]> added the comment: Yes, and I've wanted to deprecate paste.lint, but I can't because people use it over wsgiref.validate because it had this change applied. Yes, cgi.FieldStorage changed, but now that it's changed wsgiref needs to be compatible with it to be viabl

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Phillip J. Eby
Phillip J. Eby <[EMAIL PROTECTED]> added the comment: Uh, Ian, do you not remember being the person who *wrote* this code a few years ago? This is the old paste.lint with a little renaming. Of course it can be used with existing code -- code that complies with the WSGI spec. It's the cgi mo

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Ian Bicking
Ian Bicking <[EMAIL PROTECTED]> added the comment: This renders wsgiref.validate.validator completely useless, because it cannot be used with any existing code. ___ Python tracker <[EMAIL PROTECTED]> _

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Benjamin Peterson
Changes by Benjamin Peterson <[EMAIL PROTECTED]>: -- status: open -> closed ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-bugs-l

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Phillip J. Eby
Phillip J. Eby <[EMAIL PROTECTED]> added the comment: No, it shouldn't. The purpose of wsgiref.validate is to validate spec compliance, and the use of a readline() argument means that the program doing the invocation is not valid, per the spec. wsgiref.validate is correctly reporting the failur

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Benjamin Peterson
Changes by Benjamin Peterson <[EMAIL PROTECTED]>: -- assignee: -> pje nosy: +pje ___ Python tracker <[EMAIL PROTECTED]> ___ ___ Python-

[issue4330] wsgiref.validate doesn't accept arguments to readline

2008-11-15 Thread Ian Bicking
New submission from Ian Bicking <[EMAIL PROTECTED]>: The method wsgiref.validate:InputWrapper.readline doesn't take any arguments. It should take an optional size argument. Though this isn't part of the WSGI specification, the cgi module uses this argument when parsing the body, and in practice