This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author bkline
Recipients bkline
Date 2007-09-17.19:19:46
SpamBayes Score 0.00018985517
Marked as misclassified No
Message-id <1190056787.13.0.107687348277.issue1172@psf.upfronthosting.co.za>
In-reply-to
Content
I have attached a patch which adds partial documentation for the done
attribute of the cgi.FieldStorage class.  This addition is needed in
order to make it safe to rely on the current behavior of the class,
which sets this attribute to the value -1 when an uploaded file from a
CGI form does not arrive intact.

If I have sufficient free time to do so at some point in the future, I
would like to submit a more extensive modification for the module's
documentation, using a format which matches that used for most of the
other modules (that is, including documentation of all of the methods
and attributes intended for public use), and I'd be inclined to modify
the cgi.py module itself, to use a less opaque approach for conveying
the failure than the 'done' attribute.  I'd want some indication that
there was a reasonable chance that if I were to invest the work on this
improvement the results would actually be used (I see my latest patch
for the module has been languishing in the 'ignored' pile for over a
year).  If anyone else is doing some overhaul work in this area, please
speak up so we don't end up with duplication of effort.

I would think that this module would be one of the most commonly used in
the entire Python library, and would thus warrant careful maintenance
(including addressing all of the "XXX let's fix this ..." comments.

Cheers!
Files
File name Uploaded
cgi.rst.diff bkline, 2007-09-17.19:19:46
History
Date User Action Args
2007-09-17 19:19:47bklinesetspambayes_score: 0.000189855 -> 0.00018985517
recipients: + bkline
2007-09-17 19:19:47bklinesetspambayes_score: 0.000189855 -> 0.000189855
messageid: <1190056787.13.0.107687348277.issue1172@psf.upfronthosting.co.za>
2007-09-17 19:19:47bklinelinkissue1172 messages
2007-09-17 19:19:46bklinecreate