CVSS2
Attack Vector
NETWORK
Attack Complexity
LOW
Authentication
NONE
Confidentiality Impact
NONE
Integrity Impact
PARTIAL
Availability Impact
NONE
AV:N/AC:L/Au:N/C:N/I:P/A:N
CVSS3
Attack Vector
NETWORK
Attack Complexity
LOW
Privileges Required
NONE
User Interaction
NONE
Scope
CHANGED
Confidentiality Impact
NONE
Integrity Impact
LOW
Availability Impact
NONE
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:C/C:N/I:L/A:N
EPSS
Percentile
50.8%
If user-supplied input was passed into append/override_content_security_policy_directives
, a semicolon could be injected leading to directive injection.
This could be used to e.g. override a script-src
directive. Duplicate directives are ignored and the first one wins. The directives in secure_headers
are sorted alphabetically so they pretty much all come before script-src
. A previously undefined directive would receive a value even if SecureHeaders::OPT_OUT
was supplied.
The fixed versions will silently convert the semicolons to spaces and emit a deprecation warning when this happens. This will result in innocuous browser console messages if being exploited/accidentally used. In future releases, we will raise application errors resulting in 500s.
> Duplicate script-src directives detected. All but the first instance will be ignored.
See https://www.w3.org/TR/CSP3/#parse-serialized-policy
> Note: In this case, the user agent SHOULD notify developers that a duplicate directive was ignored. A console warning might be appropriate, for example.
Depending on what major version you are using, the fixed versions are 6.2.0, 5.1.0, 3.8.0.
If you are passing user input into the above methods, you could filter out the input:
override_content_security_policy_directives(:frame_src, [user_input.gsub(";", " ")])
Reported in https://github.com/twitter/secure_headers/issues/418
https://www.w3.org/TR/CSP3/#parse-serialized-policy
If you have any questions or comments about this advisory:
Vendor | Product | Version | CPE |
---|---|---|---|
secure_headers | * | cpe:2.3:a:twitter:secure_headers:*:*:*:*:*:ruby:*:* |
github.com/advisories/GHSA-xq52-rv6w-397c
github.com/rubysec/ruby-advisory-db/blob/master/gems/secure_headers/CVE-2020-5217.yml
github.com/twitter/secure_headers/commit/936a160e3e9659737a9f9eafce13eea36b5c9fa3
github.com/twitter/secure_headers/issues/418
github.com/twitter/secure_headers/pull/421
github.com/twitter/secure_headers/security/advisories/GHSA-xq52-rv6w-397c
nvd.nist.gov/vuln/detail/CVE-2020-5217
CVSS2
Attack Vector
NETWORK
Attack Complexity
LOW
Authentication
NONE
Confidentiality Impact
NONE
Integrity Impact
PARTIAL
Availability Impact
NONE
AV:N/AC:L/Au:N/C:N/I:P/A:N
CVSS3
Attack Vector
NETWORK
Attack Complexity
LOW
Privileges Required
NONE
User Interaction
NONE
Scope
CHANGED
Confidentiality Impact
NONE
Integrity Impact
LOW
Availability Impact
NONE
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:C/C:N/I:L/A:N
EPSS
Percentile
50.8%