From c67e9185b683e55275acbb8769666fc02bed31e4 Mon Sep 17 00:00:00 2001 From: Manuel Alejandro de Brito Fontes Date: Wed, 24 Jan 2018 10:01:37 -0300 Subject: [PATCH] By default brotli is disabled (#1970) --- docs/user-guide/configmap.md | 4 +++- internal/ingress/controller/config/config.go | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/docs/user-guide/configmap.md b/docs/user-guide/configmap.md index 081152e22..4695be7cf 100644 --- a/docs/user-guide/configmap.md +++ b/docs/user-guide/configmap.md @@ -448,7 +448,9 @@ The default mime type list to compress is: `application/atom+xml application/jav ## enable-brotli Enables or disables compression of HTTP responses using the ["brotli" module](https://github.com/google/ngx_brotli). -The default mime type list to compress is: `application/xml+rss application/atom+xml application/javascript application/x-javascript application/json application/rss+xml application/vnd.ms-fontobject application/x-font-ttf application/x-web-app-manifest+json application/xhtml+xml application/xml font/opentype image/svg+xml image/x-icon text/css text/plain text/x-component`. This is *enabled* by default. +The default mime type list to compress is: `application/xml+rss application/atom+xml application/javascript application/x-javascript application/json application/rss+xml application/vnd.ms-fontobject application/x-font-ttf application/x-web-app-manifest+json application/xhtml+xml application/xml font/opentype image/svg+xml image/x-icon text/css text/plain text/x-component`. This is *disabled* by default. + +*Note:* Brotli does not works in Safari < 11 https://caniuse.com/#feat=brotli ## brotli-level diff --git a/internal/ingress/controller/config/config.go b/internal/ingress/controller/config/config.go index e35ce77e7..070ce27ce 100644 --- a/internal/ingress/controller/config/config.go +++ b/internal/ingress/controller/config/config.go @@ -520,7 +520,7 @@ func NewDefault() Configuration { SSLSessionCacheSize: sslSessionCacheSize, SSLSessionTickets: true, SSLSessionTimeout: sslSessionTimeout, - EnableBrotli: true, + EnableBrotli: false, UseGzip: true, WorkerProcesses: strconv.Itoa(runtime.NumCPU()), WorkerShutdownTimeout: "10s",