Skip to content

Commit

Permalink
Merge pull request #88 from PaulThompson/update/helmetv7-peer
Browse files Browse the repository at this point in the history
Test & fix with helmet v7 and set as a peer dependency on `^6||^7`
  • Loading branch information
venables authored Nov 12, 2024
2 parents 4143936 + bf76e01 commit 5659f38
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 13 deletions.
22 changes: 12 additions & 10 deletions package-lock.json

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

4 changes: 2 additions & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,8 +26,8 @@
"engines": {
"node": ">= 14.0.0"
},
"dependencies": {
"helmet": "^6.0.1"
"peerDependencies": {
"helmet": "^6 || ^7"
},
"devDependencies": {
"ava": "^3.13.0",
Expand Down
1 change: 0 additions & 1 deletion test/koa-helmet.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,6 @@ test('it sets individual headers properly', t => {
app.use(helmet.frameguard('deny'));
app.use(helmet.noSniff());
app.use(helmet.permittedCrossDomainPolicies());
app.use(helmet.expectCt());

app.use(ctx => {
ctx.body = 'Hello world!';
Expand Down

0 comments on commit 5659f38

Please sign in to comment.