Merge pull request #97 from apinf/feature/github-auth-username

done username fetching when auth with github

Author: elijah

.meteor/packages

@@ -56,6 +56,7 @@ yogiben:admin
 timmyg:wow
 tsega:skrollr
 accounts-github
+bruz:github-api
 chart:chart
 brylie:api-umbrella
 percolate:synced-cron

.meteor/versions

@@ -20,6 +20,8 @@ blaze@2.1.2
 blaze-tools@1.0.3
 boilerplate-generator@1.0.3
 bootstrap@0.3.1
+bruz:github-api@0.2.4_1
+brylie:api-umbrella@0.2.4
 brylie:api-umbrella@0.2.3
 callback-hook@1.0.3
 cfs:access-point@0.1.49

client/startup.js

@@ -0,0 +1,5 @@
+Accounts.ui.config({
+  requestPermissions: {
+    github: ['user:email']
+  }
+});

server/accounts.coffee

@@ -11,7 +11,15 @@ Accounts.onCreateUser (options, user) ->
         user.profile.firstName = user.services.google.given_name
         user.profile.lastName = user.services.google.family_name
     if user.services?.github
-        user.emails = [{address: user.services.github.email, verified: true}]
+
+        if(user.services.github.email == null or user.services.github.email == "")
+            user.emails = [{address: "", verified: false}]
+        else
+            user.emails = [{address: user.services.github.email, verified: true}]
+
+        user.profile.name = user.services.github.username;
+
+
 
     if user.services?.facebook?.id
         profileImageUrl = 'https://graph.facebook.com/v2.3/' + user.services.facebook.id + '/picture?type=normal'
@@ -64,3 +72,25 @@ Accounts.onCreateUser (options, user) ->
     ApiUmbrellaUsers.insert(response.data.user)
 
     user
+
+# This part is still under development since there was an issue in github-accounts package
+# TODO: GitHub authentication with user's private email address
+Accounts.onLogin (info) ->
+  user = info.user
+  if user
+    github = new GitHub(
+      version: '3.0.0'
+      timeout: 5000)
+    github.authenticate
+      type: 'oauth'
+      token: user.services.github.accessToken
+    try
+      result = github.user.getEmails(user: user.services.github.username)
+      email = _(result).findWhere(primary: true)
+      ###Meteor.users.update { _id: user._id }, $set:
+        'profile.email': email.email
+        'services.github.email': email.email###
+      user.emails = [{address: email.email, verified: true}]
+      console.log user
+    catch e
+      console.log e.message