Skip to content

replacing liboneandone with liboneandone-2 with updated mocha dependency#681

Open
pfuri wants to merge 2 commits intopkgcloud:masterfrom
pfuri:master
Open

replacing liboneandone with liboneandone-2 with updated mocha dependency#681
pfuri wants to merge 2 commits intopkgcloud:masterfrom
pfuri:master

Conversation

@pfuri
Copy link

@pfuri pfuri commented Jan 28, 2020

The liboneandone package is no longer supported and contains an older version of mocha as a dependency that has a known vulnerabilities.

The latest commit of liboneandone has an update for this dependency which resolves these vulnerabilities by using an updated version of mocha, however, liboneandone hasn't been published to npm since July of 2018.

liboneandone-2 is a fork of the liboneandone package at the latest commit.

There is an open pull request for pkgcloud which addresses this through a direct reference to the GitHub commit in liboneandone, but direct references to GitHub rather than to npm modules can cause problems for organizations with proxies.

pkgcloud Issue References:

pkgcloud Pull Request References:

liboneandone Issue References:

@coveralls
Copy link

Coverage Status

Coverage remained the same at 71.391% when pulling 735c9a6 on pfuri:master into b6e5b0d on pkgcloud:master.

@KevLehman
Copy link

Are there any plans to merge this? My audits keep failing because of this dep :(

@rjherrera
Copy link

Hi! I really think this PR (or any PRs related to fixing vulnerabilities caused by old dependencies) should be merged, seems like a necessary change 🙏 ! Are there any plans of doing it? Sorry to bother @indexzero, maybe you can help us with this. Thanks!

@juanvillegas
Copy link

My audits are failing too, partially because of this.
What I don't fully grasp yet is why liboneandone doesn't declare a devDependency instead? Why is Mocha a hard dependency?

@maks-rafalko
Copy link

Same here, we have a failed build because of it. Any new on getting this solved?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants