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

[Bug] dependency System.Security.SecureString 4.3.0 reporting transient dependency vulnerabilities #4900

Open
mikegoatly opened this issue Aug 20, 2024 · 0 comments

Comments

@mikegoatly
Copy link

Library version used

4.63.0

.NET version

Compiling against 9.0.100-preview.7.24407.12

Scenario

PublicClient - desktop app

Is this a new or an existing app?

This is a new app or experiment

Issue description and reproduction steps

Nuget audit in .NET 9 has changed the default NuGetAuditMode from “direct” to “all” (Related article). This means that using the the .NET 9 SDK to build existing software is starting to highlight transient dependencies that have known vulnerabilities.

This has highlighted the use of System.Security.SecureString as it takes a transient dependency on System.Private.Uri v4.3.0:

image

This only becomes a problem when trying to use MSAL against an Android or iOS target:

image

To reproduce this, create an application that targets Android or iOS and use the .NET 9 preview SDK to build your application.

Are there any plans to remove this dependency, or otherwise mitigate this?

Relevant code snippets

-

Expected behavior

It should be possible to use MSAL with Android and iOS with the .NET 9 SDK without any warnings or errors.

Identity provider

Microsoft Entra ID (Work and School accounts and Personal Microsoft accounts)

Regression

No response

Solution and workarounds

No response

@mikegoatly mikegoatly added needs attention Delete label after triage untriaged Do not delete. Needed for Automation labels Aug 20, 2024
@bgavrilMS bgavrilMS added bug confidential-client and removed untriaged Do not delete. Needed for Automation needs attention Delete label after triage tracked-ado confidential-client labels Aug 22, 2024
@gladjohn gladjohn moved this from Committed to In Progress in MSAL Customer Trust / QM Jan 1, 2025
@gladjohn gladjohn added this to the 4.67.0 milestone Jan 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: In Progress
Development

No branches or pull requests

4 participants