SimpleAuthentication icon indicating copy to clipboard operation
SimpleAuthentication copied to clipboard

Windows Live image not showing up

Open mvneerven opened this issue 12 years ago • 5 comments

I just found out that the model.AuthenticatedClient.UserInformation.Picture property is empty when signing in with Windows Live, although it has been set in my Windows Live Profile

mvneerven avatar Jun 06 '13 22:06 mvneerven

Hey,

There's no scope for getting the users profile pircture, no way to know if the user has a picture or not.

http://msdn.microsoft.com/en-us/library/live/hh243646.aspx#accessing

You can access the users profile picture by sending a GET request to https://apis.live.net/v5.0/* user id */picture

I didn't include this because I didn't want to handle the checking to see if it was successful or not, it's not a scope we can guarantee exists.

phillip-haydon avatar Jun 07 '13 00:06 phillip-haydon

@PureKrome should we attempt to see if the image exists, and include it in the claims, and do the same with Facebook?

phillip-haydon avatar Jun 07 '13 07:06 phillip-haydon

I would say "yes please".

--- Oorspronkelijk bericht ---

Van: "Phillip Haydon" [email protected] Verzonden: 7 juni 2013 09:59 Aan: "PureKrome/WorldDomination.Web.Authentication" [email protected] CC: "mvneerven" [email protected] Onderwerp: Re: [WorldDomination.Web.Authentication] Windows Live image not showing up (#61)

@PureKrome should we attempt to see if the image exists, and include it in the claims, and do the same with Facebook?


Reply to this email directly or view it on GitHub: https://github.com/PureKrome/WorldDomination.Web.Authentication/issues/61#issuecomment-19093790

mvneerven avatar Jun 07 '13 08:06 mvneerven

@phillip-haydon i think we should, also. It's easy to use rest-sharp to do a quick 'check'. I say - lets add it for sure!

PureKrome avatar Jun 09 '13 03:06 PureKrome

Also, I need to see what happens if i try a link where the user does NOT have a pic. Is it like gravatar and sets a 'default' pic? If so, do we want to show that? If yes - then we don't need to check .. we just hardcode the Picture property value with the dynamic userId.

Definitely want to investigate it.

PureKrome avatar Jun 09 '13 03:06 PureKrome