GetUserId extension method doesn't honor IdentityOptions.ClaimsIdentity.UserIdClaimType

[brockallen]

Same for GetUserName. They're both hard coded to use the legacy SOAP/WS-* ClaimTypes enum.

[rustd]

Add this to the announcements repo.

[brockallen]

I'm guessing you're not looking for a fix, so might I suggest adding these as extension methods to the VS template project and not into the core identity framework? In many projects the display name should not be the same as the username the user enters to login.

[leastprivilege]

really? I always say "good morning MICROSOFT/PRANAV" when I meet him him the hallway.

[MaximRouiller]

I'll have to greet him that way at the next Summit. ;-)

[HaoK]

We are actually going to nuke this extension since it was only important in 2.0 because all the user manager APIs took user id, now that they take the user itself, its no longer needed

[brockallen]

yay. the less people are forced to use certain claims types, the better. thx.

[HaoK]

Gone 1c26e8f

[HaoK]

Looks like we need to provide alternate sugar for the templates in some from before removing these

[HaoK]

Going to remove all the identity claims principal extensions and add instance methods on SignInManager:

GetUserId/UserName/IsSignedIn...

Templates will need to be updated

[HaoK]

2dade94