search

stream-utils / inflation

Easily unzip an HTTP stream
MIT License
18 stars 4 forks source link

content-encoding can contain multiple values #3

Closed dougwilson closed 10 years ago

dougwilson commented 10 years ago

The content-encoding header is actually a comma-separated list. We could either support multiple encodings at once, or just assert that there is only one encoding and say we don't (yet) support multiple.

jonathanong commented 10 years ago

ya just support 1. throw on anything crazy.

dougwilson commented 10 years ago

Yea. We can probably let things like gzip, identity slide, though. I see those come by sometimes.

jonathanong commented 10 years ago

what. why? that makes no sense. maybe strip out identitys?

dougwilson commented 10 years ago

idk. It is technically correct, saying up have peel off the gzip encoding, then peel off the identity encoding (which is a no-op). It's some weird Java library that I see doing it.

dougwilson commented 10 years ago

Reading headers is moving to a different module, anyway, so this module does not care (it only supports a single encoding)