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

deps: setprototypeof@1.2.0 #81

Merged
merged 1 commit into from
Jan 24, 2020

Conversation

wesleytodd
Copy link
Member

Doesn't have impact here, but this release includes mitigation for a possible prototype pollution. I will be opening up a series of these on the Express repos.

@wesleytodd
Copy link
Member Author

Also of note, this can be ported directly onto the 2.0 branch. If you give me a LGTM I will go ahead and do the merge and push, if you want to get me publishing things I can go ahead with a publish as well. Maybe including #42 as that is ready.

Copy link
Contributor

@dougwilson dougwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @wesleytodd . Can you update HISTORY.md as well? It should look similar to 50cb805

@dougwilson dougwilson self-assigned this Jan 24, 2020
Copy link
Contributor

@dougwilson dougwilson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i updated the history file for you to get this merged

@dougwilson dougwilson merged commit a22f10e into pillarjs:master Jan 24, 2020
@wesleytodd
Copy link
Member Author

Sorry! I had left it because I just amended the commit in the 2.0.0-alpha.2 branch, but it makes sense like you said to merge here and rebase that.

@dougwilson
Copy link
Contributor

yea, otherwise we'll just have to end up cherry-picking it backwards instead of merge forwards is the only diff. it's no prob

@wesleytodd wesleytodd deleted the update-setprototypeof branch March 16, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants