Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow setting the content type for multipart uploads #272

Merged
merged 1 commit into from Jul 21, 2022
Merged

Allow setting the content type for multipart uploads #272

merged 1 commit into from Jul 21, 2022

Conversation

ns-sjorgedeaguiar
Copy link
Contributor

This is based on the patch from @chuigda, rebased onto master with
docs extended.

Fixes #172

@ns-sjorgedeaguiar
Copy link
Contributor Author

Use case: when uploading a ~1GB gzip-compressed file to a MinIO storage the file upstream ends up with a different SHA-256 hash value if I don't specify the correct content type, even though the file can be extracted normally. Setting the content type to application/gzip solves the problem for me.

@ns-sjorgedeaguiar
Copy link
Contributor Author

Hi @durch , any updates on a review for this one? Anything I can do to help?
Thanks!

@durch
Copy link
Owner

durch commented Jul 21, 2022

@ns-sjorgedeaguiar great stuff, took me a while to get to this :)

@durch durch merged commit f453fc2 into durch:master Jul 21, 2022
@ns-sjorgedeaguiar
Copy link
Contributor Author

@durch thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add an API put_object_stream_with_content_type
2 participants