Merge pull request github-tools#185 from ele828/master

Ændrew Rininsland · Ændrew Rininsland · commit 3c5dd961d0b0 · 2015-05-26T21:19:57.000-06:00
Support Chinese file names. Thanks @ele828!
diff --git a/github.js b/github.js
@@ -521,6 +521,7 @@
       // --------
 
       this.contents = function(ref, path, cb) {
+        path = encodeURI(path);
         _request("GET", repoPath + "/contents" + (path ? "/" + path : ""), { ref: ref }, cb);
       };
 
@@ -595,7 +596,7 @@
       // -------
 
       this.read = function(branch, path, cb) {
-        _request("GET", repoPath + "/contents/"+path, {ref: branch}, function(err, obj) {
+        _request("GET", repoPath + "/contents/"+encodeURI(path), {ref: branch}, function(err, obj) {
           if (err && err.error === 404) return cb("not found", null, null);
 
           if (err) return cb(err);
@@ -663,9 +664,9 @@
       // -------
 
       this.write = function(branch, path, content, message, cb) {
-        that.getSha(branch, path, function(err, sha) {
+        that.getSha(branch, encodeURI(path), function(err, sha) {
           if (err && err.error !== 404) return cb(err);
-          _request("PUT", repoPath + "/contents/" + path, {
+          _request("PUT", repoPath + "/contents/" + encodeURI(path), {
             message: message,
             content: btoa(content),
             branch: branch,
diff --git a/test/test.repo.js b/test/test.repo.js
@@ -103,6 +103,13 @@ test('Create Repo', function(t) {
       q.end();
     });
   });
+
+  t.test('repo.writeChinese', function(q) {
+    repo.write('master', '中文测试.md', 'THIS IS A TEST', 'Creating test', function(err) {
+      q.error(err);
+      q.end();
+    });
+  });
   clearTimeout(timeout);
   t.end();
 });
