Skip to content

upgrade lodash and types/lodash#907

Merged
gustavohenke merged 1 commit intoexpress-validator:masterfrom
liqiang372:upgrade-lodash
Aug 2, 2020
Merged

upgrade lodash and types/lodash#907
gustavohenke merged 1 commit intoexpress-validator:masterfrom
liqiang372:upgrade-lodash

Conversation

@liqiang372
Copy link
Copy Markdown
Contributor

upgrade due to security concern. #901

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 100.0% when pulling ee4bf9f on liqiang372:upgrade-lodash into 210e0ca on express-validator:master.

@liqiang372
Copy link
Copy Markdown
Contributor Author

Hi @gustavohenke, this pr is pending for a week, not sure what's the plan for this and other pending prs. Since I don't have pr merging and issues management permissions. Do you want to find some time to discuss the future of this project?

@gustavohenke
Copy link
Copy Markdown
Member

Oh sorry for this one.
I thought of letting you merge it 🤦 I would like to see some more PRs/code reviews before being comfortable with giving write access.

@gustavohenke gustavohenke merged commit 8e61cbe into express-validator:master Aug 2, 2020
@gustavohenke
Copy link
Copy Markdown
Member

Published as v6.6.1.

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.

Update lodash to >=4.17.19 to fix vulnerability

3 participants