Use authenticated name and email when committing

fix #128

Author: iamdanfox

github.js

@@ -407,22 +407,26 @@
       // -------
 
       this.commit = function(parent, tree, message, cb) {
-        var data = {
-          "message": message,
-          "author": {
-            "name": options.user
-          },
-          "parents": [
-            parent
-          ],
-          "tree": tree
-        };
-
-        _request("POST", repoPath + "/git/commits", data, function(err, res) {
+        var user = new Github.User();
+        user.show(null, function(err, userData){
           if (err) return cb(err);
-          currentTree.sha = res.sha; // update latest commit
-          cb(null, res.sha);
-        });
+          var data = {
+            "message": message,
+            "author": {
+              "name": options.user,
+              "email": userData.email
+            },
+            "parents": [
+              parent
+            ],
+            "tree": tree
+          };
+          _request("POST", repoPath + "/git/commits", data, function(err, res) {
+            if (err) return cb(err);
+            currentTree.sha = res.sha; // update latest commit
+            cb(null, res.sha);
+          });
+        })
       };
 
       // Update the reference of your head to point to the new commit SHA

test/commit_test.js

@@ -19,7 +19,11 @@ describe("Repo", function(){
   it("should return commit errors correctly", function(done){
     var repo = github.getRepo(TEST_USERNAME, TEST_REPO);
     repo.commit("broken-parent-hash", "broken-tree-hash", "commit message", function(err){
-      done();
+      if (err != null && err.request != null && err.request.status == 500) {
+        done();
+      } else {
+        done(new Error("No `err` passed to .commit callback"));
+      }
     });
   });