Set properties on the context before calling OnAuthenticated so they are available to the callback (#145)

Chris Patrick · Tratcher · commit 6fcee9515f43 · 2017-11-30T07:07:28.000-08:00
Addresses #143
diff --git a/src/Microsoft.Owin.Security.MicrosoftAccount/MicrosoftAccountAuthenticationHandler.cs b/src/Microsoft.Owin.Security.MicrosoftAccount/MicrosoftAccountAuthenticationHandler.cs
@@ -121,10 +121,10 @@ protected override async Task<AuthenticationTicket> AuthenticateCoreAsync()
                     context.Identity.AddClaim(new Claim(ClaimTypes.Email, context.Email, "http://www.w3.org/2001/XMLSchema#string", Options.AuthenticationType));
                 }
 
-                await Options.Provider.Authenticated(context);
-
                 context.Properties = properties;
 
+                await Options.Provider.Authenticated(context);
+
                 return new AuthenticationTicket(context.Identity, context.Properties);
             }
             catch (Exception ex)

Original file line number	Diff line number	Diff line change
`@@ -121,10 +121,10 @@ protected override async Task<AuthenticationTicket> AuthenticateCoreAsync()`
`121`	`121`	`context.Identity.AddClaim(new Claim(ClaimTypes.Email, context.Email, "http://www.w3.org/2001/XMLSchema#string", Options.AuthenticationType));`
`122`	`122`	`}`
`123`	`123`
`124`		`- await Options.Provider.Authenticated(context);`
`125`		`-`
`126`	`124`	`context.Properties = properties;`
`127`	`125`
	`126`	`+ await Options.Provider.Authenticated(context);`
	`127`	`+`
`128`	`128`	`return new AuthenticationTicket(context.Identity, context.Properties);`
`129`	`129`	`}`
`130`	`130`	`catch (Exception ex)`