Pull requests

#46 Merged
Repository
gavinwahl gavinwahl
Branch
default
Repository
david david
Branch
default

Ungzip on download based on Content-Encoding, not arbitrary Content-Type

Author
  1. Gavin Wahl
Reviewers
Description

When getting a file from S3, it should be gunzipped based on it's Content-Encoding. It was using the guessed content type.

Comments (3)

  1. David Larlet repo owner

    Hello,

    Agreed with the fix, why did you compare directly to "gzip" and not self._storage.gzip_content_types anymore?

    Thanks for your contribution, David

  2. Gavin Wahl author

    It's looking at Content-Encoding, not Content-Type. gzip_content_types is a list of content types that should be gzipped, not the gzip content encoding itself.