Store extra oauth user data as extended data on the userinfo object about oauth2

[ejsmith]

+1

[ArtsevDmitry]

For example, need phone, but this property not presented in UserInfo :)

[jordisan]

We are using your fantastic library , but now we need to get additional user info from LinkedIn; concretely, the country of the user (inside location): https://developer.linkedin.com/docs/fields/basic-profile

Note that location is a structured data item, so maybe key-value pairs is not the best solution.

What do you recommend us? Extending your library somehow? Thanks

[niemyjski]

Previously in a different library, we always parsed the data into key value pairs. Currently this functionality isn't available (myself or someone needs to build it) but we should have a dictionary where we can return all the extended data items.

[niemyjski]

In this case you could parse it out into location.name = value or you could store it as json?

[jordisan]

Maybe we could add a AllData member to UserInfo class... (maybe XDocument or JObject); and then store there all the user data coming from the server

[ejsmith]

I would think you would want to store the values in a dictionary and not use something that is transport specific like jobject and xdoc.

[jordisan]

Sure, @ejsmith ... I was just thinking about a quick-and-dirty solution :)

[weihangChen]

I got this working in a quick way, without modifying existing API

using System.Collections.Generic;
using System.Linq;
using OAuth2.Configuration;
using OAuth2.Infrastructure;
using OAuth2.Models;
using OAuth2.Client;

namespace OAuth2.Example.Models
{
    public class MyGoogleClient : OAuth2.Client.Impl.GoogleClient
    {

        public MyGoogleClient(IRequestFactory factory, IClientConfiguration configuration)
            : base(factory, configuration)
        {
        }


        protected override UserInfo ParseUserInfo(string content)
        {
            return base.ParseUserInfo(content);
        }

    }

    public class MyAuthorizationRoot : AuthorizationRoot
    {
        private readonly IRequestFactory _requestFactory;
        private readonly OAuth2ConfigurationSection _configurationSection;

        public MyAuthorizationRoot() :
            this(new ConfigurationManager(), "oauth2", new RequestFactory())
        {
        }

        public MyAuthorizationRoot(
           IConfigurationManager configurationManager,
           string configurationSectionName,
           IRequestFactory requestFactory)
        {
            _requestFactory = requestFactory;
            _configurationSection = configurationManager
                .GetConfigSection<OAuth2ConfigurationSection>(configurationSectionName);
        }

        public override IEnumerable<IClient> Clients
        {
            get
            {
                var clients = base.Clients.ToList();
                var googleClient = clients.FirstOrDefault(x => x.Name == "Google");
                var newGoogleClient = new MyGoogleClient(_requestFactory, googleClient.Configuration);

                clients.Remove(googleClient);
                clients.Add(newGoogleClient);

                return clients;
            }
        }


    }
}

[niemyjski]

I don't see where you are storing the extra info? I think we should add a dictionary on the user info and stick it in there. Would anyone like to take a stab at this?