Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

safer response data parsing in response handlers and added user.getOr…

…gs(). Fixes #37. Fixes #36
  • Loading branch information...
commit 8299cedb8f8c6713d922bea961796604bc878d33 1 parent 7abed8a
@mikedeboer mikedeboer authored
View
20 api/v3.0.0/events.js
@@ -37,7 +37,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -79,7 +79,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -121,7 +121,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -163,7 +163,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -204,7 +204,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -245,7 +245,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -286,7 +286,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -327,7 +327,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -368,7 +368,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -410,7 +410,7 @@ var events = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
24 api/v3.0.0/gists.js
@@ -37,7 +37,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -78,7 +78,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -119,7 +119,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -160,7 +160,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -199,7 +199,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -238,7 +238,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -277,7 +277,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -316,7 +316,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -355,7 +355,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -394,7 +394,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -433,7 +433,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -472,7 +472,7 @@ var gists = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
29 api/v3.0.0/gitdata.js
@@ -40,7 +40,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -82,7 +82,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -123,7 +123,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -168,7 +168,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -209,7 +209,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -251,7 +251,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -293,7 +293,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -336,7 +336,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -377,7 +377,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -418,7 +418,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -465,7 +465,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -507,7 +507,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -538,7 +538,8 @@ var gitdata = module.exports = {
*
* - user (String): Required.
* - repo (String): Required.
- * - tree (Json): Required.
+ * - tree (Json): Required. Array of Hash objects (of path, mode, type and sha) specifying a tree structure
+ * - base_tree (String): Optional. String of the SHA1 of the tree you want to update with new data
**/
this.createTree = function(msg, block, callback) {
var self = this;
@@ -548,7 +549,7 @@ var gitdata = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
5 api/v3.0.0/gitdataTest.js
@@ -44,7 +44,7 @@ var test = module.exports = {
);
},
- "test: POST repos/:user/:repo/git/blobs (createBlob)": function(next) {
+ "test: POST /repos/:user/:repo/git/blobs (createBlob)": function(next) {
var self = this;
this.client.authenticate({
type: "token",
@@ -291,7 +291,8 @@ var test = module.exports = {
{
user: "String",
repo: "String",
- tree: "Json"
+ tree: "Json",
+ base_tree: "String"
},
function(err, res) {
Assert.equal(err, null);
View
44 api/v3.0.0/issues.js
@@ -43,7 +43,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -93,7 +93,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -134,7 +134,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -179,7 +179,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -225,7 +225,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -268,7 +268,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -309,7 +309,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -351,7 +351,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -393,7 +393,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -434,7 +434,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -477,7 +477,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -519,7 +519,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -560,7 +560,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -600,7 +600,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -641,7 +641,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -683,7 +683,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -725,7 +725,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -769,7 +769,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -810,7 +810,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -854,7 +854,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -899,7 +899,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -940,7 +940,7 @@ var issues = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
46 api/v3.0.0/orgs.js
@@ -38,7 +38,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -79,7 +79,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -123,7 +123,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -164,7 +164,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -204,7 +204,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -244,7 +244,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -283,7 +283,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -323,7 +323,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -363,7 +363,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -403,7 +403,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -442,7 +442,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -481,7 +481,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -523,7 +523,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -564,7 +564,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -603,7 +603,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -644,7 +644,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -684,7 +684,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -724,7 +724,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -764,7 +764,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -805,7 +805,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -846,7 +846,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -887,7 +887,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -928,7 +928,7 @@ var orgs = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
30 api/v3.0.0/pullRequests.js
@@ -40,7 +40,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -81,7 +81,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -125,7 +125,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -168,7 +168,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -212,7 +212,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -255,7 +255,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -298,7 +298,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -341,7 +341,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -383,7 +383,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -426,7 +426,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -467,7 +467,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -512,7 +512,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -555,7 +555,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -597,7 +597,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -638,7 +638,7 @@ var pullRequests = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
94 api/v3.0.0/repos.js
@@ -38,7 +38,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -80,7 +80,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -122,7 +122,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -167,7 +167,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -214,7 +214,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -254,7 +254,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -301,7 +301,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -344,7 +344,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -386,7 +386,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -428,7 +428,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -470,7 +470,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -512,7 +512,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -554,7 +554,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -595,7 +595,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -636,7 +636,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -677,7 +677,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -721,7 +721,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -762,7 +762,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -804,7 +804,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -847,7 +847,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -893,7 +893,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -934,7 +934,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -976,7 +976,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1018,7 +1018,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1059,7 +1059,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1101,7 +1101,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1142,7 +1142,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1183,7 +1183,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1226,7 +1226,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1267,7 +1267,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1309,7 +1309,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1350,7 +1350,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1392,7 +1392,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1435,7 +1435,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1476,7 +1476,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1518,7 +1518,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1558,7 +1558,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1599,7 +1599,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1641,7 +1641,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1681,7 +1681,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1721,7 +1721,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1763,7 +1763,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1804,7 +1804,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1848,7 +1848,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1895,7 +1895,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1936,7 +1936,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -1977,7 +1977,7 @@ var repos = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
9 api/v3.0.0/routes.json
@@ -2201,6 +2201,15 @@
}
},
+ "get-orgs": {
+ "url": "/user/orgs",
+ "method": "GET",
+ "params": {
+ "$page": null,
+ "$per_page": null
+ }
+ },
+
"get-emails": {
"url": "/user/emails",
"method": "GET",
View
74 api/v3.0.0/user.js
@@ -36,7 +36,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -75,7 +75,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -120,7 +120,47 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
+ }
+ catch (ex) {
+ if (callback)
+ callback(new error.InternalServerError(ex.message), res);
+ return;
+ }
+
+ if (!ret)
+ ret = {};
+ if (!ret.meta)
+ ret.meta = {};
+ ["x-ratelimit-limit", "x-ratelimit-remaining", "link"].forEach(function(header) {
+ if (res.headers[header])
+ ret.meta[header] = res.headers[header];
+ });
+
+ if (callback)
+ callback(null, ret);
+ });
+ };
+
+ /** section: github
+ * user#getOrgs(msg, callback) -> null
+ * - msg (Object): Object that contains the parameters and their values to be sent to the server.
+ * - callback (Function): function to call when the request is finished with an error as first argument and result data as second argument.
+ *
+ * ##### Params on the `msg` object:
+ *
+ * - page (Number): Optional. Page number of the results to fetch. Validation rule: ` ^[0-9]+$ `.
+ * - per_page (Number): Optional. A custom page size up to 100. Default is 30. Validation rule: ` ^[0-9]+$ `.
+ **/
+ this.getOrgs = function(msg, block, callback) {
+ var self = this;
+ this.client.httpSend(msg, block, function(err, res) {
+ if (err)
+ return self.sendError(err, null, msg, callback);
+
+ var ret;
+ try {
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -160,7 +200,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -199,7 +239,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -238,7 +278,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -279,7 +319,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -320,7 +360,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -360,7 +400,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -401,7 +441,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -440,7 +480,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -479,7 +519,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -519,7 +559,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -558,7 +598,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -598,7 +638,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -639,7 +679,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
@@ -678,7 +718,7 @@ var user = module.exports = {
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
View
19 api/v3.0.0/userTest.js
@@ -80,6 +80,25 @@ var test = module.exports = {
);
},
+ "test: GET /user/orgs (getOrgs)": function(next) {
+ var self = this;
+ this.client.authenticate({
+ type: "token",
+ username: username,
+ token: token
+ });
+ this.client.user.getOrgs(
+ {
+ page: "Number",
+ per_page: "Number"
+ },
+ function(err, res) {
+ Assert.equal(err, null);
+ // other assertions go here
+ }
+ );
+ },
+
"test: GET /user/emails (getEmails)": function(next) {
var self = this;
this.client.authenticate({
View
2  index.js
@@ -242,7 +242,7 @@ var Client = module.exports = function(config) {
def = paramsStruct[paramName];
value = trim(msg[paramName]);
- if (typeof value !== 'boolean' && !value) {
+ if (typeof value != "boolean" && !value) {
// we don't need to validation for undefined parameter values
// that are not required.
if (!def.required)
View
2  templates/handler.js.tpl
@@ -7,7 +7,7 @@
var ret;
try {
- ret = JSON.parse(res.data);
+ ret = res.data && JSON.parse(res.data);
}
catch (ex) {
if (callback)
Please sign in to comment.
Something went wrong with that request. Please try again.