Merge pull request hapijs#254 from AdriVanHoudt/google-profile-fix

fix mapping of google profile fixes hapijs#251
PierreGabioud · Aug 21, 2016 · 837052e · 837052e
2 parents 56c4bf8 + b1cb92a
commit 837052e
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/providers/google.js b/lib/providers/google.js
@@ -13,7 +13,7 @@ exports = module.exports = function (options) {
             get('https://www.googleapis.com/oauth2/v3/userinfo', null, (profile) => {
 
                 credentials.profile = {
-                    id: profile.id,
+                    id: profile.sub,
                     displayName: profile.name,
                     name: {
                         given_name: profile.given_name,

diff --git a/test/providers/google.js b/test/providers/google.js
@@ -35,7 +35,7 @@ describe('google', () => {
                 Hoek.merge(custom, provider);
 
                 const profile = {
-                    id: '1234567890',
+                    sub: '1234567890',
                     name: 'steve smith',
                     given_name: 'steve',
                     family_name: 'smith',