Misleading comments in django.http.multipartparser.py
|Reported by:||KayEss||Owned by:||Gary Wilson|
|Cc:||Triage Stage:||Ready for checkin|
|Has patch:||yes||Needs documentation:||no|
|Needs tests:||no||Patch needs improvement:||no|
There are a number of comments in the file presumably left over from before the refactoring work. The patch simply removes some and rewords a few others.
- The input data is not a bytestring, it's a file. See my web site for a StringIO example.
- file_upload_dir isn't mentioned anywhere apart from in the comment. I guess that the upload handlers are now meant to deal with this.
The upload parser does also seem to be missing some error handling. It doesn't appear to check that it has parsed the same number of bytes as the specified content length and it certainly doesn't throw any sort of error if the input data is empty.
The patch is against r8047.