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

High severity vulnerabilities identified with the dependency package used in Moq 4.16.1 #1169

Closed
Rashmi-nw opened this issue Jun 8, 2021 · 4 comments

Comments

@Rashmi-nw
Copy link

We are using Moq@4.16.1 which has a dependency on Castle.Core@4.4.0 . This has a dependency on NETStandard.Library@1.6.1.

NETStandard.Library@1.6.1 has a dependency on System.Text.RegularExpressions@4.3.0 which has a high severity vulnerability.

Info on High severity vulnerability : Description: Regular Expression Denial of Service (ReDoS) - https://snyk.io/vuln/SNYK-DOTNET-SYSTEMTEXTREGULAREXPRESSIONS-174708

@stakx
Copy link
Contributor

stakx commented Jun 8, 2021

Thanks for the notification. Our (Moq's) hands are bound however, since there currently isn't any newer version of Castle.Core targeting a more recent .NET Standard.

Work is currently underway at https://github.com/castleproject/Core towards a new major release that will target .NET Standard 2.x, though that release hasn't happened yet.

I suppose in the meantime, you could try manually adding a dependency on a newer version of the System.Text.RegularExpressions package.

/cc @jonorossi (who manages the Castle.Core release process)

@Rashmi-nw
Copy link
Author

Thanks @stakx - I've manually added a dependency to use the latest version of System.Text.RegularExpressions.That has resolved the issue for now. And thanks for the update. :)

@stakx
Copy link
Contributor

stakx commented Jun 8, 2021

@Rashmi-nw, since you found a solution, I am going to close this issue. The only course of action for us will be to update our dependency on Castle.Core, and that'll happen anyway as soon as it becomes available.

@stakx stakx closed this as completed Jun 8, 2021
@jonorossi
Copy link

I suppose in the meantime, you could try manually adding a dependency on a newer version of the System.Text.RegularExpressions package.

Sounds like a good solution. Castle.Core does not use the System.Text.RegularExpressions package so it won't break anything.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants