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.

Sign up for GitHub

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

Jump to bottom

Calling ConfClientApp::AcquireTokenForClientAsync with a certificate on .NET 4.7 throws an exception #448

Closed
mjcheetham opened this issue Jul 31, 2017 · 0 comments
Closed

Calling ConfClientApp::AcquireTokenForClientAsync with a certificate on .NET 4.7 throws an exception #448

mjcheetham opened this issue Jul 31, 2017 · 0 comments
Assignees
@trwalke
Milestone
Preview 1.1.1

Comments

@mjcheetham
Copy link
Contributor

When calling AcquireTokenForClientAsync with an IClientAssertionCertificate from an application on .NET Framework 4.7 an exception is thrown with inner exception "Object reference not set to an instance of an object."
On 4.6.2 works correctly and no exception is thrown.

This is the same issue encountered on ADAL.NET (as it's the same code). The default RSA provider which is returned from X509AsymmetricSecurityKey::GetAsymmetricAlgorithm in .NET Framework 4.7 has changed.

Please see this bug:
AzureAD/azure-activedirectory-library-for-dotnet#708.

And this PR:
AzureAD/azure-activedirectory-library-for-dotnet#732
@mjcheetham mjcheetham mentioned this issue Jul 31, 2017
Closed
@kpanwar kpanwar added this to the Preview 1.1.1 milestone Aug 7, 2017
trwalke pushed a commit that referenced this issue Aug 11, 2017
Fix #448: Handle new RSA default (RSACng) on netfx4.7
6be6497
@trwalke trwalke mentioned this issue Aug 15, 2017
Merged
trwalke referenced this issue Aug 15, 2017
@trwalke
Merge branch 'dev' into trwalke/HandleNewRSACngOnNetfx4.7-#448
6efeddc
trwalke referenced this issue Aug 15, 2017
Merge remote-tracking branch 'origin/trwalke/HandleNewRSACngOnNetfx4.…
addf393 
…7-#448' into trwalke/HandleNewRSACngOnNetfx4.7-#448
trwalke referenced this issue Aug 16, 2017
@trwalke
Merge branch 'dev' into trwalke/HandleNewRSACngOnNetfx4.7-#448
14722fb
trwalke added a commit that referenced this issue Aug 16, 2017
@trwalke
Trwalke/handle new rsa cng on netfx4.7 #448 (#454)
4bb204f 
* Fix #448: Handle new RSA default (RSACng) on netfx4.7

* Added unit test to test cryptography libraries used by MSAL.

* Fixed Unit test

* Updated testCert to use an RSA private key
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@trwalke trwalke

Labels
None yet
Projects
None yet
Milestone
Preview 1.1.1
Development

No branches or pull requests
3 participants
@mjcheetham @kpanwar @trwalke