Skip to content

Commit d3560e0

Browse files
authored
replace accepts with negotiator (#197)
2 parents 5e6ee06 + 1e95836 commit d3560e0

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

Diff for: index.js

+3-8
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@
1414
* @private
1515
*/
1616

17-
var accepts = require('accepts')
17+
var Negotiator = require('negotiator')
1818
var Buffer = require('safe-buffer').Buffer
1919
var bytes = require('bytes')
2020
var compressible = require('compressible')
@@ -174,13 +174,8 @@ function compression (options) {
174174
}
175175

176176
// compression method
177-
var accept = accepts(req)
178-
var method = accept.encoding(['gzip', 'deflate', 'identity'])
179-
180-
// we really don't prefer deflate
181-
if (method === 'deflate' && accept.encoding(['gzip'])) {
182-
method = accept.encoding(['gzip', 'identity'])
183-
}
177+
var negotiator = new Negotiator(req)
178+
var method = negotiator.encoding(['gzip', 'deflate', 'identity'], ['gzip'])
184179

185180
// negotiation failed
186181
if (!method || method === 'identity') {

Diff for: package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@
99
"license": "MIT",
1010
"repository": "expressjs/compression",
1111
"dependencies": {
12-
"accepts": "~1.3.8",
1312
"bytes": "3.1.2",
1413
"compressible": "~2.0.18",
1514
"debug": "2.6.9",
15+
"negotiator": "~0.6.4",
1616
"on-headers": "~1.0.2",
1717
"safe-buffer": "5.2.1",
1818
"vary": "~1.1.2"

0 commit comments

Comments
 (0)