[Bug] Code Cleanup in Microsoft.Identity.Web.UI AccountController

[jmprieur]

Which Version of Microsoft Identity Web are you using ?
Microsoft Identity Web 0.1.5-preview

Where is the issue?

• Web App
  • [x ] Sign-in users
  • Sign-in users and call web APIs

Suggestions

• AccountController uses IOptionsMonitor whereas the rest of Microsoft.Identity.Web.UI uses IOptions. The recommendation is to use IOptions
• AccountController sets explicitly a scheme value whereas Chris recommends to just use null (which challenges the default provider)

Note
The AzureAD.UI counterpart does the same as Microsoft.Identity.Web.UI's account controller.

[jmprieur]

@Tratcher FYI

[Tratcher]

• AccountController sets explicitly a scheme value whereas Chris recommends to just use null (which challenges the default provider)

I checked on this and found that the auth scheme is only required to be non-null for SignIn. I've filed dotnet/aspnetcore#23325 to fix that. You can remove the fallback for all of the others.

[jennyf19]

Included in 0.2.0-preview release