Skip to content

Security Issue Flagged by Snyk #10698

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

Closed
gregorymey-dev opened this issue Mar 17, 2021 · 8 comments
Closed

Security Issue Flagged by Snyk #10698

gregorymey-dev opened this issue Mar 17, 2021 · 8 comments

Comments

@gregorymey-dev
Copy link

Hi Guys,

Thanks for the awesome tool.

Could you investigate the vulnerabilities that Snyk flagged.

✗ Arbitrary Code Injection [Medium Severity][https://snyk.io/vuln/SNYK-JS-EJS-1049328] in [email protected]
introduced by [email protected] > [email protected] > [email protected] > @surma/[email protected] > [email protected]
This issue was fixed in versions: 3.1.6
✗ Regular Expression Denial of Service (ReDoS) [Medium Severity][https://snyk.io/vuln/SNYK-JS-GLOBPARENT-1016905] in [email protected]
introduced by [email protected] > [email protected] > [email protected] > [email protected] > [email protected] > [email protected]
This issue was fixed in versions: 5.1.2
✗ Regular Expression Denial of Service (ReDoS) [Medium Severity][https://snyk.io/vuln/SNYK-JS-HTMLPARSESTRINGIFY2-1079307] in [email protected]
introduced by [email protected] > [email protected]
No upgrade or patch available
✗ Regular Expression Denial of Service (ReDoS) [Medium Severity][https://snyk.io/vuln/SNYK-JS-ISSVG-1085627] in [email protected]
introduced by [email protected] > [email protected] > [email protected] > [email protected] > [email protected] > [email protected]
This issue was fixed in versions: 4.2.2
✗ Regular Expression Denial of Service (ReDoS) [High Severity][https://snyk.io/vuln/SNYK-JS-SSRI-1085630] in [email protected]
introduced by [email protected] > [email protected] > [email protected] > [email protected] > [email protected]
This issue was fixed in versions: 8.0.1

@candrews
Copy link

See https://snyk.io/vuln/SNYK-JS-SSRI-1085630

This issue is a duplicate of #10699

@carbonphyber
Copy link

Note that the first comment is a dump of multiple security reports and the second only describes 1 and refers to another GH Issue about that 1 security issue. Issue #10699 is only a partial duplicate of #10698 (this Issue).

Linkifying the SNYK security reports:

This was referenced Mar 23, 2021
@gregorymey-dev
Copy link
Author

Note that the first comment is a dump of multiple security reports and the second only describes 1 and refers to another GH Issue about that 1 security issue. Issue #10699 is only a partial duplicate of #10698 (this Issue).

Linkifying the SNYK security reports:

Thanks, I've noticed I accidently copied [email protected] into the message, this section should not be there.

@nj314
Copy link

nj314 commented Apr 26, 2021

@gregorymey-dev This issue needs to be reopened as there is no other issue covering the [email protected] vuln, which currently does not have a patch version available, so react-scripts needs changes to produce an upgrade to ejs@^3.1.6. As stated by @carbonphyber , this issue is only a partial duplicate of #10699 . We need this issue to capture the necessary major version upgrade from ejs 2 to 3.

@gusliedke
Copy link

Indeed. This still needs to be fixed. We need to upgrade [email protected] to [email protected].

Arbitrary Code Injection
Affected module: [email protected]
Exploit maturity: Proof of Concept
Fixed in: [email protected]

[email protected][email protected][email protected]  ›  @surma/[email protected][email protected]```

@senkevichdv
Copy link

Any updates on this issue? I see this issue is still closed

Arbitrary Code Injection
Affected module: [email protected]
Introduced through: [email protected]
Exploit maturity: Proof of Concept
Fixed in: [email protected]

Introduced through:   [email protected][email protected][email protected]  ›  @surma/[email protected][email protected]

@gregorymey-dev
Copy link
Author

Hi Guys,

Was out of circulation for a while, IO would want to re-open this case.

@gaearon
Copy link
Contributor

gaearon commented Jul 7, 2021

None of these issues affect how CRA uses these dependencies. There is nothing to address here.
#11174

@gaearon gaearon closed this as completed Jul 7, 2021
@facebook facebook locked as resolved and limited conversation to collaborators Jul 7, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants