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

fix: use new forked package of json-pointer that fixed critical Prototype pollution #133

Merged
merged 5 commits into from
Feb 21, 2022

Conversation

francis04j
Copy link
Contributor

What/Why/How?

+--------------+------------------+----------+-------------------+---------------+---------------------------------------+
| LIBRARY | VULNERABILITY ID | SEVERITY | INSTALLED VERSION | FIXED VERSION | TITLE |
+--------------+------------------+----------+-------------------+---------------+---------------------------------------+
| json-pointer | CVE-2021-23820 | CRITICAL | 0.6.1 | | Prototype Pollution in json-pointer |
| | | | | | -->avd.aquasec.com/nvd/cve-2021-23820 |
+--------------+------------------+----------+-------------------+---------------+---------------------------------------+

Reference

GHSA-v5vg-g7rq-363w

@MichaelMcAleer
Copy link

This vulnerability has been flagged by our internal security platforms and code scanners, it would be great if this PR could go in ASAP! Thanks @francis04j for opening this request!

@RomanHotsiy
Copy link
Member

@francis04j I can't find any changes in your forked repo. This is somehow concerned as it means you published to npm from local.

Could you push you changes?

@francis04j
Copy link
Contributor Author

@francis04j I can't find any changes in your forked repo. This is somehow concerned as it means you published to npm from local.

Could you push you changes?

@RomanHotsiy changs pushed

@RomanHotsiy
Copy link
Member

Thanks, we'll merge and release it soon.

package.json Outdated Show resolved Hide resolved
@RomanHotsiy RomanHotsiy merged commit 8c971a8 into Redocly:master Feb 21, 2022
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.

4 participants