defnull / multipart

Multipart parser for Python 3
Other
120 stars 33 forks source link

multipart: empty payload support implemented #20

Closed bryzgaloff closed 4 years ago

bryzgaloff commented 4 years ago

Added support for the empty multipart payload. Here is an example of a valid empty payload:

--theboundary--\r\n

All the tests are passed. Reach out to me if any additional tests are required.

With this addition multipart package works perfectly for me.

Please approve and publish a new version to PyPI. Thank you for the useful package!

defnull commented 4 years ago

Good catch, thanks. Could you add a small test for this error case to prevent regressions? Then I'll gladly merge and release.

bryzgaloff commented 4 years ago

@defnull here it is :) master branch fails on the test while the patched version does not

defnull commented 4 years ago

Merged and published. Thanks

bryzgaloff commented 4 years ago

Maybe you should have to add a tagged release to releases.