Skip to content
This repository has been archived by the owner on Jul 24, 2019. It is now read-only.

Dependencies update : fix security issues #732

Merged
merged 1 commit into from
Aug 10, 2017
Merged

Dependencies update : fix security issues #732

merged 1 commit into from
Aug 10, 2017

Conversation

Ilshidur
Copy link
Contributor

@Ilshidur Ilshidur commented Aug 2, 2017

The 2.1.14 version has some vulnerabilities that can be fixed by updating the npm dependencies.

This PR fixes :

  • 1 Uninitialized Memory Exposure : updated extract-zip@1.5.0 to version 1.6.5
  • 1 Uninitialized Memory Exposure : updated request@2.79.0 to version 2.81.0 (change log)

Fixes : #728

The 2.1.14 version [has some vulnerabilities](https://snyk.io/test/npm/phantomjs-prebuilt/2.1.14) that can be fixed by updating the npm dependencies.

This PR fixes :
* 1 *Uninitialized Memory Exposure* : updated `extract-zip@1.5.0` to version _1.6.5_
* 1 *Uninitialized Memory Exposure* : updated `request@2.79.0` to version _2.81.0_ ([change log](https://github.com/request/request/blob/master/CHANGELOG.md#v2810-20170309))
@Ilshidur Ilshidur mentioned this pull request Aug 2, 2017
@avindra
Copy link
Contributor

avindra commented Aug 5, 2017

@nicks Can you please have a look? Services that catch security vulnerabilities like VersionEye prevent us from using projects that depend on phantomjs because of this issue.

@avindra
Copy link
Contributor

avindra commented Aug 10, 2017

@adrianlee44 @eduardoramirez @jfuchs

Can one of you have a look? 🙏

or tag the appropriate person to handle merging this fix

@jfuchs jfuchs merged commit 65b57f7 into Medium:master Aug 10, 2017
@avindra
Copy link
Contributor

avindra commented Aug 11, 2017

Thank you for merging this @jfuchs ! Can you please also bump the patch version number in package.json so that a release is cut?

I can make a PR for that if you like.

@jfuchs
Copy link
Contributor

jfuchs commented Aug 11, 2017

That'd be great, yes. (As you might've guessed, this isn't under active development, sorry).

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.

Update dependencies
3 participants