Re: Consider reverting multipart parsing change

[Joshua Ballanco <jballanc@gmail.com>]

On Jul 6, 2010, at 12:54 AM, Jeremy Kemper wrote:

> On Mon, Jul 5, 2010 at 3:57 PM, Mark Goris <mark.goris@gmail.com> wrote:
>> It's been a long time since I created the initial ticket, so I've lost
>> some of the context (especially in the subsequent discussion that led
>> to this change that you've suggested to revert).  It seemed best to me
>> that tempfiles only be used in the case of having a filename attribute
>> be present, regardless of content-type.  Your suggestion seems to
>> focus on the specific type of content-type driving logic regarding
>> whether or not a part is treated as a file upload.  I guess my
>> question is, why not just use the filename attribute to drive this
>> logic?  I don't know of a downside to this approach (meaning, I'm not
>> exactly clear on the intention for the change that prompted ticket 79
>> in the first place).
> 
> Checking for filename works for file uploads from web browsers. All
> other MIME parts are flattened to a "normal" form parameter even it it
> was neither a form param or a file upload. For example, see the
> multipart/mixed regression on #79.
> 
> The fix for #79 restricts what may be considered a file upload rather
> than broadening what may be considered a normal form param to include
> MIME parts with a text/plain Content-Type.
> 
> Rather than treating all MIME parts as form params and making a
> special case for file uploads by looking for a filename, we should be
> doing the opposite, making a special case to identify form param MIME
> parts by their missing filename and missing or text/plain
> Content-Type.

Just wanted to give my +1 to this solution. I actually have a real-life use case where this becomes an issue. I have a client (not a web browser) which generates data dynamically to upload to an HTTP server as a file upload. Because there was no original file from which to extract a filename parameter (and because the server has logic to assign filenames based on other parameters), my client does a multipart-upload without a filename parameter. However, the file parts can be rather large (up to 100s of MB). Having these passed in as strings instead of using tempfiles seems less than ideal.

Cheers,

Josh