MicrosoftAccountAuthenticationProvider.OnAuthenticated does not have access to properties

[chrispatrick]

I'm not sure if there's a reason for this behaviour, but I'm using the MicrosoftAccountAuthenticationMiddleware and implementing a custom handler in OnAuthenticated. However, context.Properties is null at this point which means I can't maintain any custom state across the authentication process without doing something like manually creating and deleting cookies.

It appears to be a pretty simple thing in MicrosoftAccountAuthenticationHandler.cs to swap lines 124 and 126 around so that the properties are set on the context before OnAuthenticated is called.

[Tratcher]

This looks like a typo. All of the other providers set the Properties first.

AspNetKatana/src/Microsoft.Owin.Security.MicrosoftAccount/MicrosoftAccountAuthenticationHandler.cs

Lines 124 to 128 in 1653abe

	await Options.Provider.Authenticated(context);
	context.Properties = properties;
	return new AuthenticationTicket(context.Identity, context.Properties);

AspNetKatana/src/Microsoft.Owin.Security.Facebook/FacebookAuthenticationHandler.cs

Lines 148 to 152 in 1653abe

	context.Properties = properties;
	await Options.Provider.Authenticated(context);
	return new AuthenticationTicket(context.Identity, context.Properties);

Feel free to send a PR if you get to it before I do.

[chrispatrick]

Thanks. Thought it might be a typo. PR submitted.