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

[Med] Snyk: Denial of Service (DoS) (due 6/2/19) #2792

Closed
Tracked by #137 ...
fec-jli opened this issue Apr 3, 2019 · 4 comments · Fixed by #3003
Closed
Tracked by #137 ...

[Med] Snyk: Denial of Service (DoS) (due 6/2/19) #2792

fec-jli opened this issue Apr 3, 2019 · 4 comments · Fixed by #3003
Assignees
Labels
High priority Security: moderate Remediate within 60 days
Milestone

Comments

@fec-jli
Copy link
Contributor

fec-jli commented Apr 3, 2019

package.json 2 medium
Denial of Service (DoS) --MEDIUM SEVERITY
Vulnerable module: mem
Introduced through: npm@6.8.0
https://app.snyk.io/vuln/npm:mem:20180117

@jason-upchurch jason-upchurch added the Security: moderate Remediate within 60 days label May 16, 2019
@jason-upchurch jason-upchurch added this to the Sprint 9.1 milestone May 16, 2019
@jason-upchurch jason-upchurch changed the title Denial of Service (DoS) --MEDIUM SEVERITY in package.json (from check logs) [Med] Snyk: Denial of Service (DoS) (due 6/2/19) May 16, 2019
@jason-upchurch
Copy link
Contributor

jason-upchurch commented May 16, 2019

Denial of Service (DoS)

Vulnerable module: mem
Introduced through: npm@6.8.0

Detailed paths and remediation

  • Introduced through: fec-cms@1.0.0 › npm@6.8.0 › libnpx@10.2.0 › yargs@11.0.0 › os-locale@2.1.0 › mem@1.1.0
    Remediation: No remediation path available.
    Vulnerable functions
    index.module.exports.memoized()

Overview

mem is an optimization used to speed up consecutive function calls by caching the result of calls with identical input.

Affected versions of this package are vulnerable to Denial of Service (DoS). Old results were deleted from the cache and could cause a memory leak.

details

Denial of Service (DoS) describes a family of attacks, all aimed at making a system inaccessible to its intended and legitimate users.

Unlike other vulnerabilities, DoS attacks usually do not aim at breaching security. Rather, they are focused on making websites and services unavailable to genuine users resulting in downtime.

One popular Denial of Service vulnerability is DDoS (a Distributed Denial of Service), an attack that attempts to clog network pipes to the system by generating a large volume of traffic from many machines.

When it comes to open source libraries, DoS vulnerabilities allow attackers to trigger such a crash or crippling of the service by using a flaw either in the application code or from the use of open source libraries.

Two common types of DoS vulnerabilities:

High CPU/Memory Consumption- An attacker sending crafted requests that could cause the system to take a disproportionate amount of time to process. For example, commons-fileupload:commons-fileupload.

Crash - An attacker sending crafted requests that could cause the system to crash. For Example, npm ws package

Remediation

Upgrade mem to version 4.0.0 or higher.

@rfultz
Copy link
Contributor

rfultz commented May 29, 2019

Current status:
BLOCKED BY zkat/npx#206

I'm watching libnpx for new releases.

Updating to mem@^4.0.0 is easier said than done.

os-locale had an issue for this (sindresorhus/os-locale#32) but there was a breaking change in mem@3 which would require os-locale to jump to 3 instead of a 2.1.1. An os-locale 3 would require an update from yargs.

yargs #1209 says that yargs 12.0.2 passes the Snyk test.

But libnpx@10.2.0 wants yargs@^11.0.0. There's an issue to address this, but it hasn't received any attention. There was a previous ticket for other dependencies but it hasn't cleared this Snyk alert.

npm 6.9.0 wants libnpx@^10.2.0. If libnpx stays at 10.* we should be ready when libnpx is updated. Otherwise, we may need an update from npm to allow a currently-non-existent libnpx 11.

@dorothyyeager
Copy link
Contributor

Still blocked. Moving to 9.2

@dorothyyeager
Copy link
Contributor

Moving to 9.4; PR needs review.

@dorothyyeager dorothyyeager removed this from the Sprint 9.3 milestone Jul 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
High priority Security: moderate Remediate within 60 days
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants