fix prevent crash from user migrated profiles

midvik · Aug 15, 2015 · dfb85e4 · dfb85e4
1 parent 40ab7d2
commit dfb85e4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/common/models/User-Identity.js b/common/models/User-Identity.js
@@ -158,14 +158,14 @@ export default function(UserIdent) {
         userChanged = true;
       }
 
-      if (!(/github/).test(provider)) {
+      if (!(/github/).test(provider) && profile) {
         debug('setting social', provider, (/github/g).test(provider));
         debug('profile username', profile.username);
         user[provider] = profile.username;
       }
 
       // if user signed in with github refresh their info
-      if (/github/.test(provider)) {
+      if (/github/.test(provider) && profile && profile._json) {
         debug("user isn't github cool or username from github is different");
         setProfileFromGithub(user, profile, profile._json);
         userChanged = true;