Merge pull request #1062 from adamk33n3r/update-passport-auth-api

fix(oauth): update google/fb passport

Author: Awk34

Diff for: app/templates/server/auth(auth)/facebook(facebookAuth)/passport.js

@@ -5,7 +5,11 @@ exports.setup = function(User, config) {
   passport.use(new FacebookStrategy({
     clientID: config.facebook.clientID,
     clientSecret: config.facebook.clientSecret,
-    callbackURL: config.facebook.callbackURL
+    callbackURL: config.facebook.callbackURL,
+    profileFields: [
+      'displayName',
+      'emails'
+    ]
   },
   function(accessToken, refreshToken, profile, done) {
     <% if (filters.mongooseModels) { %>User.findOneAsync({<% }
@@ -19,7 +23,6 @@ exports.setup = function(User, config) {
             name: profile.displayName,
             email: profile.emails[0].value,
             role: 'user',
-            username: profile.username,
             provider: 'facebook',
             facebook: profile._json
           });

Diff for: app/templates/server/auth(auth)/google(googleAuth)/index.js

@@ -10,8 +10,8 @@ router
   .get('/', passport.authenticate('google', {
     failureRedirect: '/signup',
     scope: [
-      'https://www.googleapis.com/auth/userinfo.profile',
-      'https://www.googleapis.com/auth/userinfo.email'
+      'profile',
+      'email'
     ],
     session: false
   }))

Diff for: app/templates/server/auth(auth)/google(googleAuth)/passport.js

@@ -19,7 +19,7 @@ exports.setup = function(User, config) {
             name: profile.displayName,
             email: profile.emails[0].value,
             role: 'user',
-            username: profile.username,
+            username: profile.emails[0].value.split('@')[0],
             provider: 'google',
             google: profile._json
           });