[D7] Disable brotli compression of pre-compressed CSS and JS

I had Brotli enabled all my servers but never ran into this problem because I was compressing everything on the fly and didn't use pre-compressed gzip files (additional disk access didn't outweigh compression overhead).

To provide more information to others, this problem occurs only if you serve pre-compressed Brotli or Gzip files with a rewrite rule. Enabling Brotli with a filter (for example with AddOutputFilterByType BROTLI_COMPRESS application/json) does not double-encode the response.

If you serve .gz files with a rewrite rule, the patch fixes it. I call we can mark this RTBC if a few others can confirm.

The patch worked for me - thanks.

Btw advagg allows it, so better to add support for it to core

Patch from #2 works perfectly and solved it equally to Drupal 8. We should get this in as long as Drupal 7 is still maintained. Thank you all!

Here you can find the committed Drupal 8 patch / issue: #2960808: Disable brotli compression of pre-compressed CSS and JS

LGTM.

Would be good to combine this in the same release as any other .htaccess changes we're working on (e.g. #2768921: Backport server configuration code from SA-CORE-2016-003 to Drupal 7)

RTBC + 1

Thankingyou!