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

Guessing of ContentType #67

Closed
markunsworth opened this issue Jun 3, 2016 · 1 comment · May be fixed by #68
Closed

Guessing of ContentType #67

markunsworth opened this issue Jun 3, 2016 · 1 comment · May be fixed by #68

Comments

@markunsworth
Copy link

markunsworth commented Jun 3, 2016

When syncing a directory tree to S3 (using sync/dsync) the ContentType is being set as binary/octet-stream on all files.

s4cmd --num-threads=100 --recursive dsync local_app_folder/static/dist/ s3:/my_s3_bucket/app/static/ --API-ACL="public-read" --API-CacheControl="max-age=31536000, public"

As the directory structure contains varying filetypes its not possible to set using the --API-ContentType parameter. I can set the --API-ContentType to a blank string but that actually sets the contentType as blank on S3 which has unexpected behaviour when trying to load some file formats.

s4cmd --num-threads=100 --recursive dsync local_app_folder/static/dist/ s3:/my_s3_bucket/app/static/ --API-ACL="public-read" --API-CacheControl="max-age=31536000, public" --API-ContentType=""

It would be good to support the mimetype guessing like S3cmd

@markunsworth markunsworth changed the title Default ContentType Guessing of ContentType Jun 3, 2016
@markunsworth markunsworth closed this as not planned Won't fix, can't repro, duplicate, stale Oct 11, 2023
@shapirus
Copy link

shapirus commented Sep 2, 2024

what was the reason behind closing this as not planned?
is there any chance that the mime type guessing is finally merged?

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

Successfully merging a pull request may close this issue.

3 participants