Including Content-Type header in S3 signature

TkTech / ckanext-cloudstorage

Implements support for resource storage against multiple popular providers via apache-libcloud (S3, Azure Storage, etc...)

MIT License

35 stars 55 forks source link

Including Content-Type header in S3 signature #29

Closed jeverling closed 6 years ago

jeverling commented 6 years ago

Fixes #28 I didn't include any other headers, because I don't want to risk breaking anything. With this fix, the text-view preview works for JSON/GeoJSON files stored on S3, that previously wouldn't work because the signature was calculated without the Content-Type header that is sent by the JavaScript code.

TkTech commented 6 years ago

Thank you! A major update to ckanext-cloudstorage is about to be finished, however I verified this was an issue even in the new version. Going to merge this for now for users who need an immediate fix.