skoruba / Duende.IdentityServer.Admin

The administration for the Duende IdentityServer and Asp.Net Core Identity ⚡
Apache License 2.0
549 stars 194 forks source link

Update EnableAuthenticatorViewModel.cs #125

Open albrookesplowman opened 1 year ago

albrookesplowman commented 1 year ago

Changed the SharedKey and AuthenticatorUri variables to be nullable so that the MFA works from Microsoft authenticator apps.

toby-freemarket commented 1 year ago

@albrookesplowman can I ask why? I don't seem to have problems with the Microsoft authenticator app.

albrookesplowman commented 1 year ago

@toby-freemarket - I found the Microsoft authenticator app didn't work when I ran it. However, I was running on .Net 7 with the latest version of VS2022 so it might simply be related to my build environment. If it isn't a problem for anyone else then I am happy to keep these modifications specific to my build.