On Mon, Jan 18, 2021 at 01:40:44PM +0100, Tim Düsterhus wrote: > 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 >
Thanks to both of you, applied. -- William Lallemand
