Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(deps): bump helmet from 3.23.3 to 4.0.0 #347

Merged
merged 2 commits into from
Aug 7, 2020

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Aug 3, 2020

Bumps helmet from 3.23.3 to 4.0.0.

Changelog

Sourced from helmet's changelog.

4.0.0 - 2020-08-02

Added

  • helmet.contentSecurityPolicy:
    • If no default-src directive is supplied, an error is thrown
    • Directive lists can be any iterable, not just arrays

Changed

  • This package no longer has dependencies. This should have no effect on end users, other than speeding up installation time.
  • helmet.contentSecurityPolicy:
    • There is now a default set of directives if none are supplied
    • Duplicate keys now throw an error. See helmetjs/csp#73
    • This middleware is more lenient, allowing more directive names or values
  • helmet.xssFilter now disables the buggy XSS filter by default. See #230

Removed

  • Dropped support for old Node versions. Node 10+ is now required
  • helmet.featurePolicy. If you still need it, use the feature-policy package on npm.
  • helmet.hpkp. If you still need it, use the hpkp package on npm.
  • helmet.noCache. If you still need it, use the nocache package on npm.
  • helmet.contentSecurityPolicy:
    • Removed browser sniffing (including the browserSniff and disableAndroid parameters). See See helmetjs/csp#97
    • Removed conditional support. This includes directive functions and support for a function as the reportOnly. Read this if you need help.
    • Removed a lot of checks—you should be checking your CSP with a different tool
    • Removed support for legacy headers (and therefore the setAllHeaders parameter). Read this if you need help.
    • Removed the loose option
  • helmet.frameguard:
  • helmet.hidePoweredBy no longer accepts arguments. See this article to see how to replicate the removed behavior. See #224.
  • helmet.hsts:
  • helmet.xssFilter no longer accepts options. Read "How to disable blocking with X–XSS–Protection" and "How to enable the report directive with X–XSS–Protection" if you need the legacy behavior.
Commits
  • bdb0934 4.0.0
  • 926ef63 Prepare 4.0.0 release
  • 6d3f028 Update changelog for 4.0.0 release
  • 4fbf5bd Update devDependencies to latest versions
  • c9926db Remove homepage field from middleware package.jsons
  • 83508e8 4.0.0-rc.2
  • 54525e5 Export HelmetOptions type
  • c813f91 4.0.0-rc.1
  • fc2f745 Use angle brackets around links in error messages
  • e657b0b Move "simple" middleware functions inline
  • Additional commits viewable in compare view

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added the dependencies Pull requests that update a dependency file label Aug 3, 2020
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/helmet-4.0.0 branch from 6a4704e to 69de501 Compare August 4, 2020 09:58
@LoneRifle
Copy link
Contributor

This change will disable X-XSS-Protection. This is a non-standard header supported only by IE and Safari. Content-Security-Policy is the standard equivalent, and most modern browsers have disabled their XSS auditor given that there are known ways to bypass them.

Justification for the change can be found at helmetjs/helmet#230 - tl;dr - enabling this is ineffective

liangyuanruo
liangyuanruo previously approved these changes Aug 6, 2020
Copy link
Contributor

@liangyuanruo liangyuanruo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, please test extensively if you haven't, especially on the more obscure features like GA on the transition page. the changelog is long; but i didn't identify anything obvious that would break.

@LoneRifle LoneRifle dismissed liangyuanruo’s stale review August 6, 2020 03:11

Testing revealed helmet overriding custom CSP

@LoneRifle LoneRifle force-pushed the dependabot/npm_and_yarn/helmet-4.0.0 branch 3 times, most recently from 27ba630 to 50d0f24 Compare August 6, 2020 03:31
@LoneRifle LoneRifle force-pushed the dependabot/npm_and_yarn/helmet-4.0.0 branch from 50d0f24 to 5953f95 Compare August 6, 2020 04:04
@LoneRifle LoneRifle merged commit b84616d into develop Aug 7, 2020
@LoneRifle LoneRifle deleted the dependabot/npm_and_yarn/helmet-4.0.0 branch August 7, 2020 05:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants