Copy over the profile picture from Facebook?

[grenager]

Hi, it would be really helpful for me to get the profile picture from Facebook. Here's the required code change (I think):

const query = {
                appsecret_proof: Crypto.createHmac('sha256', this.clientSecret).update(credentials.token).digest('hex'),
                fields: 'id,name,email,first_name,last_name,middle_name,gender,link,locale,timezone,updated_time,verified,picture'
            };

            get('https://graph.facebook.com/v2.3/me', query, (profile) => {

                credentials.profile = {
                    id: profile.id,
                    username: profile.username,
                    displayName: profile.name,
                    name: {
                        first: profile.first_name,
                        last: profile.last_name,
                        middle: profile.middle_name
                    },
                    email: profile.email,
                    picture: profile.picture,
                    raw: profile
                };

                return callback();
            });

Better yet, it would be cool to allow users to specify the paths of the fields they want to copy out of the profiles into the session objects.

[pankajpatel]

you can access the picture by the raw as raw.picture.

raw contains the JSON profile response received.

[grenager]

Yes, but picture is not there in the raw JSON, because the code doesn't request that field from Facebook. See my proposed code changes. Thanks! Teg

…

On Tue, Feb 21, 2017 at 5:48 AM, Pankaj Patel ***@***.***> wrote: you can access the picture by the raw as raw.profile.picture. raw contains the JSON profile response received. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#297 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMm2OX-4_uW0GimM07-xjP7_ulopfXpbks5reus9gaJpZM4ME7-i> .

[pankajpatel]

@grenager You are right; though there would be many fields which facebook provides which can be included in the query.

I think it will be good to have an option for the provider to accept query param for the default request.

[grenager]

Great idea, thanks Pankaj.

…

On Mon, Feb 27, 2017 at 12:54 PM, Pankaj Patel ***@***.***> wrote: @grenager <https://github.com/grenager> You are right; though there would be many fields which facebook provides which can be included in the query. I think it will be good to have an option for the provider to accept query param for the default request. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#297 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMm2OXAxr8YUktJ7Em8HeKVkHR-Xnq7jks5rgzftgaJpZM4ME7-i> .

[ricktbaker]

Any update on this? Is there any way to make it so it requests the profile image as well? I see we can directly modify the core provider code to add this, but that isn't a particularly great solution.

[pankajpatel]

@ricktbaker PR needs to be submitted for this to make profile picture available as well.

[PaulMougel]

I just submitted a PR to fix this issue, feedback and comments are welcome 👍

You will be able to specify the fields you want to retrieve from the user's profile, which will be available in credentials.profile.raw.

[lock]

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.