Skip to content

Commit

Permalink
Merge pull request github#15331 from github/repo-sync
Browse files Browse the repository at this point in the history
repo sync
  • Loading branch information
Octomerger authored Feb 10, 2022
2 parents 6c5d0a5 + 5c9a83b commit f85f403
Show file tree
Hide file tree
Showing 4 changed files with 0 additions and 101 deletions.
2 changes: 0 additions & 2 deletions middleware/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ import csp from './csp.js'
import cookieParser from './cookie-parser.js'
import csrf from './csrf.js'
import handleCsrfErrors from './handle-csrf-errors.js'
import compression from 'compression'
import { setDefaultFastlySurrogateKey } from './set-fastly-surrogate-key.js'
import setFastlyCacheHeaders from './set-fastly-cache-headers.js'
import catchBadAcceptLanguage from './catch-bad-accept-language.js'
Expand Down Expand Up @@ -208,7 +207,6 @@ export default function (app) {

// *** Headers ***
app.set('etag', false) // We will manage our own ETags if desired
app.use(compression())
app.use(catchBadAcceptLanguage)

// *** Config and context for redirects ***
Expand Down
92 changes: 0 additions & 92 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@
"ajv-formats": "^2.1.1",
"cheerio": "^1.0.0-rc.10",
"classnames": "^2.3.1",
"compression": "^1.7.4",
"connect-datadog": "0.0.9",
"connect-slashes": "^1.4.0",
"cookie-parser": "^1.4.6",
Expand Down
6 changes: 0 additions & 6 deletions tests/rendering/server.js
Original file line number Diff line number Diff line change
Expand Up @@ -103,12 +103,6 @@ describe('server', () => {
})
})

test('uses gzip compression', async () => {
const res = await get('/en')
expect(res.headers['content-encoding']).toBe('gzip')
expect(res.headers['transfer-encoding']).toBe('chunked')
})

test('sets Content Security Policy (CSP) headers', async () => {
const res = await get('/en')
expect('content-security-policy' in res.headers).toBe(true)
Expand Down

0 comments on commit f85f403

Please sign in to comment.