theriverman / django-minio-backend

Minio Backend for Django

https://pypi.org/project/django-minio-backend/

MIT License

111 stars 22 forks source link

Returning content bytes on MinioBackend._save #28

Closed MarcelFox closed 2 years ago

MarcelFox commented 2 years ago

Info

I was facing an issue were the content length was not matching the content size, because the content was being interpreted as empty:

OSError at /my-fancy-endpoint
stream having not enough data;expected: 666, got: 0 bytes

Issue occurs if I upload files as iso, text, pdf and so forth.
Minio SDK Version: 7.1.0

This MR is also related to the closed issue #21.

theriverman commented 2 years ago

Thanks for the fix, @MarcelFox ! It's looking good, so I'm merging it to develop. Soon I'll release a new version including your fix too, but I need a little more time for verification.