Merge pull request #688 from codydaig/bugfix/github_undefined_display…

…name If displayName in GitHub is undefined use username Fixes #519
meanjs · Jul 24, 2015 · 8a81cf0 · 8a81cf0
2 parents 3a39f0a + 7605956
commit 8a81cf0
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/modules/users/server/config/strategies/github.js b/modules/users/server/config/strategies/github.js
@@ -22,7 +22,7 @@ module.exports = function(config) {
 			providerData.refreshToken = refreshToken;
 
 			// Create the user OAuth profile
-			var displayName = profile.displayName.trim();
+			var displayName = profile.displayName ? profile.displayName.trim() : profile.username.trim();
 			var iSpace = displayName.indexOf(' '); // index of the whitespace following the firstName
 			var firstName =  iSpace !== -1 ? displayName.substring(0, iSpace) : displayName;
 			var lastName = iSpace !== -1 ? displayName.substring(iSpace + 1) : '';