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

Upgrade dependency pkg version to address security vulnerability. #5193

Closed
wants to merge 1 commit into from

Conversation

imcarolwang
Copy link
Contributor

@imcarolwang imcarolwang commented Jul 7, 2023

For #5190.
The vulnerabilities are also reported in ci pipeline.

Cc: @mconnew , @HongGit

@HongGit
Copy link
Contributor

HongGit commented Jul 7, 2023

@imcarolwang we will wait for a secure version of System.Security.Cryptography.Xml.

@imcarolwang imcarolwang marked this pull request as draft July 7, 2023 23:55
@MoSelem
Copy link

MoSelem commented Jul 12, 2023

@imcarolwang we will wait for a secure version of System.Security.Cryptography.Xml.

is 7.0.1 not a secure version?

@imcarolwang
Copy link
Contributor Author

@imcarolwang we will wait for a secure version of System.Security.Cryptography.Xml.

is 7.0.1 not a secure version?

It appears that 7.0.1 version of System.Security.Cryptography.Xml relies on System.Security.Cryptography.Pkcs (>= 7.0.0) , and 7.0.0 version of System.Security.Cryptography.Pkcs has the same severity vulnerability. So we still need to wait.

@MoSelem
Copy link

MoSelem commented Jul 12, 2023

I think the vulnerability exists in version 7.0.0 for only Net7,
WCF is releasing for Net6 only so far, and the vulnerability would be in Pks >=6.0.0 and <= 6.0.2, and the patched version is 6.0.3,
so maybe version 7.0.0 for Net6 is fine?

@MoSelem
Copy link

MoSelem commented Jul 13, 2023

I think the vulnerability exists in version 7.0.0 for only Net7, WCF is releasing for Net6 only so far, and the vulnerability would be in Pks >=6.0.0 and <= 6.0.2, and the patched version is 6.0.3, so maybe version 7.0.0 for Net6 is fine?

@imcarolwang

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.

3 participants