NameIDentifier should use Id from context instead of username (#232)

woutervs · TerribleDev · commit f60e8b5ffaa7 · 2018-04-01T02:25:11.000-04:00
diff --git a/src/Owin.Security.Providers.Discord/DiscordAuthenticationHandler.cs b/src/Owin.Security.Providers.Discord/DiscordAuthenticationHandler.cs
@@ -105,7 +105,7 @@ protected override async Task<AuthenticationTicket> AuthenticateCoreAsync()
                 };
                 if (!string.IsNullOrEmpty(context.Id))
                 {
-                    context.Identity.AddClaim(new Claim(ClaimTypes.NameIdentifier, context.UserName, XmlSchemaString, Options.AuthenticationType));
+                    context.Identity.AddClaim(new Claim(ClaimTypes.NameIdentifier, context.Id, XmlSchemaString, Options.AuthenticationType));
                 }
                 if (!string.IsNullOrEmpty(context.UserName))
                 {

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ protected override async Task<AuthenticationTicket> AuthenticateCoreAsync()`
`105`	`105`	`};`
`106`	`106`	`if (!string.IsNullOrEmpty(context.Id))`
`107`	`107`	`{`
`108`		`- context.Identity.AddClaim(new Claim(ClaimTypes.NameIdentifier, context.UserName, XmlSchemaString, Options.AuthenticationType));`
	`108`	`+ context.Identity.AddClaim(new Claim(ClaimTypes.NameIdentifier, context.Id, XmlSchemaString, Options.AuthenticationType));`
`109`	`109`	`}`
`110`	`110`	`if (!string.IsNullOrEmpty(context.UserName))`
`111`	`111`	`{`