Author vstinner
Recipients amaury.forgeotdarc, barry, eric.araujo, erob, flox, ggenellina, oopos, pebbe, pitrou, quentel, r.david.murray, tcourbon, tercero12, tobias, v+python, vstinner
Date 2011-01-10.21:28:55
SpamBayes Score 5.10024e-07
Marked as misclassified No
Message-id <1294694940.22.0.320017538896.issue4953@psf.upfronthosting.co.za>
In-reply-to
Content
> I set an attribute encoding to self.fp because, for each part 
> of a multipart/form-data, a new instance of FieldStorage is created,
> and this instance needs to know how to decode bytes.

Set fp.encoding may raise an error (eg. for a read-only object, or an object implemented in C). You should add a new argument to the constructor.

> Maybe I'm missing something here, but sys.stdin is always
> a TextIOWrapper instance, even if set to binary mode

I mean: you should pass sys.stdin.buffer instead of sys.stdin.
History
Date User Action Args
2011-01-10 21:29:00vstinnersetrecipients: + vstinner, barry, amaury.forgeotdarc, ggenellina, pitrou, eric.araujo, v+python, r.david.murray, oopos, tercero12, tcourbon, tobias, flox, pebbe, quentel, erob
2011-01-10 21:29:00vstinnersetmessageid: <1294694940.22.0.320017538896.issue4953@psf.upfronthosting.co.za>
2011-01-10 21:28:55vstinnerlinkissue4953 messages
2011-01-10 21:28:55vstinnercreate