Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Fix: Avoids prototype pollution #7

Merged
merged 1 commit into from
Mar 18, 2021
Merged

Fix: Avoids prototype pollution #7

merged 1 commit into from
Mar 18, 2021

Conversation

sttk
Copy link
Contributor

@sttk sttk commented Mar 13, 2021

This pr is to fix the vulnerability of prototype pollution.

phated
phated approved these changes Mar 17, 2021
View reviewed changes
Copy link
Member

@phated phated left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@sttk sorry for the delay reviewing this. It looks great! Feel free to merge and release it! 🎉

I also noticed that I'm not a publisher on this package (and maybe others) on npm. Will you add me?

@sttk sttk merged commit 2c738f5 into gulpjs:master Mar 18, 2021
@sttk
Copy link
Contributor Author

sttk commented Mar 18, 2021

Thank you for approving. I merged this pr.

@sttk sttk deleted the fix/avoid-prototype-pollution branch March 18, 2021 13:07
@sttk
Copy link
Contributor Author

sttk commented Mar 18, 2021

@phated I checked that you are already in maintainers on npm about the packages I transfered.

@debricked debricked bot mentioned this pull request Jul 1, 2021
Open
@debricked debricked bot mentioned this pull request Feb 23, 2023
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@phated phated phated approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sttk @phated