AzureAD/microsoft-identity-abstractions-for-dotnet

[Bug] CertificateDescription from path with passwords reads the password for both the path and the password

jmprieur opened this issue · 0 comments

What?
Having this JSON content

"ClientCredentials": [
            {
                "SourceType": "Path",
                "CertificateDiskPath": "C:\\myCert.pfx",
                "CertificatePassword": "something here"
            }

and then reading the certificate description, this will read "something here" for both the CertificateDiskPath and CertificatePassword

Analysis?
This is because of the Container/ReferenceOrValue calculated properties that are not consistent with each other in this case

Suggestion
Write the unit test for these as well (even if they are only used for testing)