[vlc-devel] [PATCH] vlc_stream/http: add GET_CONTENT_ENCODING

Rémi Denis-Courmont remi at remlab.net
Thu Jul 6 19:56:47 CEST 2017


Le torstaina 6. heinäkuuta 2017, 19.46.20 EEST Francois Cartegnie a écrit :
> Because there's no way to tell if the content
> is a natively compressed file, or has been compressed
> by http. (for conditional insertion of deflate filter)

The distinction does not even make sense in HTTP. As such, the patch cannot 
solve that problem, and indeed, client side cannot solve that problem. 
Besides, it is highly questionable why the client should even distinguish 
those cases.

Specifically, you cannot distinguish between a resource whose only 
representation is in compresssed form, and a resource that has both compressed 
and uncompressed representation. Some servers expose both cases in the same 
way, with the Content-Encoding header (or not).

So regardless of the merits and dismerits of the patch, the commit message 
looks totally wrong to me.

-- 
雷米‧德尼-库尔蒙
https://www.remlab.net/



More information about the vlc-devel mailing list