Skip to content
This repository has been archived by the owner on Nov 19, 2023. It is now read-only.

Vuln #40

Closed
wants to merge 11 commits into from
Closed

Vuln #40

wants to merge 11 commits into from

Conversation

peterennis
Copy link
Contributor

There a 5 security issues in the master branch.
Updated deps. One of them is alpha.
Using npm 6.1.0
npm audit - after latest updates reports 0 vulns

I used npm test and it errored.
I do not know enough about the tests to take it further.

I would like to test this further. Is there a simple app to try?

capture330

@gms1
Copy link
Owner

gms1 commented Jun 28, 2018

First of all, thank you for your efforts!
I've already looked at this problem a while ago and it's not that easy to fix, see my comments in #41
But I would like to change the build procedure for this package, so that this problem is then fixed at the latest

@peterennis
Copy link
Contributor Author

Outdated.

@peterennis peterennis closed this Jun 29, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants