Remi,

Am 18.01.21 um 11:24 schrieb Remi Tricot-Le Breton:
> The patches look good to me. Just one point concerning the first one,
> you could have removed everything related to ACCEPT_ENCODING_MAX_ENTRIES
> since the limit was induced by the array that does not exist anymore.

I have intentionally opted to leave it as is, as a bit of a safety net.
It is not going to make sense to send more than 16 *known* encodings at
once. This limits processing time for an abusive client that sends:

accept-encoding: br,br,br,br,[…],br

> The comment of the accept_encoding_normalizer function does not match
> its behavior anymore either.

Indeed. I adjusted that on v2.

Best regards
Tim Düsterhus


Reply via email to