fix: lowercase accept-encoding header (#43)

Sébastien Chopin · Pooya Parsa · commit 454807c5ab8f · 2019-05-21T22:36:47.000+04:30
+ update docs
diff --git a/docs/api/readme.md b/docs/api/readme.md
@@ -126,7 +126,7 @@ When directing requests at a url protected by CloudFlare's CDN you should set th
 
 ### `proxyHeadersIgnore`
 
-* Default `['host', 'accept']`
+* Default `['accept', 'host', 'cf-ray', 'cf-connecting-ip']`
 
 Only efficient when `proxyHeaders` is set to true. Removes unwanted request headers to the API backend in SSR.
 
diff --git a/lib/plugin.js b/lib/plugin.js
@@ -108,7 +108,7 @@ export default (ctx, inject) => {
 
   if (process.server) {
     // Don't accept brotli encoding because Node can't parse it
-    defaults.headers['Accept-Encoding'] = 'gzip, deflate'
+    defaults.headers['accept-encoding'] = 'gzip, deflate'
   }
 
   // Create new HTTP instance
diff --git a/test/fixture/pages/ssr.vue b/test/fixture/pages/ssr.vue
@@ -16,7 +16,7 @@ export default {
     },
 
     httpEncoding() {
-      return this.$http._defaults.headers['Accept-Encoding']
+      return this.$http._defaults.headers['accept-encoding']
     }
   },
   fetch({ app, route }) {

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ export default (ctx, inject) => {`
`108`	`108`
`109`	`109`	`if (process.server) {`
`110`	`110`	`// Don't accept brotli encoding because Node can't parse it`
`111`		`- defaults.headers['Accept-Encoding'] = 'gzip, deflate'`
	`111`	`+ defaults.headers['accept-encoding'] = 'gzip, deflate'`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`// Create new HTTP instance`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ export default {`
`16`	`16`	`},`
`17`	`17`
`18`	`18`	`httpEncoding() {`
`19`		`- return this.$http._defaults.headers['Accept-Encoding']`
	`19`	`+ return this.$http._defaults.headers['accept-encoding']`
`20`	`20`	`}`
`21`	`21`	`},`
`22`	`22`	`fetch({ app, route }) {`