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

Dependency updates (for solving security issues) #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

nicky1038
Copy link

Please see #9 for details.

nicky1038 and others added 3 commits February 5, 2019 16:20
Lodash versions <=4.17.4 were vulnerable to Prototype Pollution, for details please see:
https://www.npmjs.com/advisories/577

For changes in Lodash 4.x please see:
https://github.com/lodash/lodash/releases/tag/4.0.0
https://github.com/lodash/lodash/wiki/Changelog#compatibility-warnings

Only this change was needed to pay attention to:
* Removed _.pluck in favor of _.map with iteratee shorthand

Here are other changes related to methods used in font-loader but there is no need to change anything in code:
* Split _.assign & _.assignIn into _.assignWith & _.assignInWith
* Split _.invert into _.invertBy
* Split _.omit & _.pick into _.omitBy & _.pickBy
* Split _.uniq into _.sortedUniq, _.sortedUniqBy, & _.uniqBy
* Made “By” methods like _.groupBy & _.sortBy provide a single param to iteratees
* Removed multiValue param from _.invert
* Removed legacy options param signature from _.template
* Enabled shortcut fusion for _.find
According to the change logs, there are no API changes in these packages thus only package.json can be edited:
https://github.com/fontello/ttf2eot/blob/master/CHANGELOG.md
https://github.com/fontello/ttf2woff/blob/master/CHANGELOG.md
Though vulnerability https://nodesecurity.io/advisories/577 was fixed in Lodash 4.17.5, versions <=4.17.11 remained vulnerable to Prototype Pollution, for details please see:
https://www.npmjs.com/advisories/1065
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.

1 participant