Skip to content

Commit d07598d

Browse files
Merge pull request #20 from Microsoft/users/chjohn/node-error
Adding basic logging of VSO exception messages on failed requests
2 parents d5a10bc + c3bf4ce commit d07598d

2 files changed

Lines changed: 15 additions & 2 deletions

File tree

api/RestClient.ts

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,20 @@ export function processResponse(url, res, contents, serializationData: Serializa
5151
if (res.statusCode > 299) {
5252
// not success
5353
var msg = httpCodes[res.statusCode] ? "Failed Request: " + httpCodes[res.statusCode] : "Failed Request";
54-
msg += '(' + res.statusCode + ') - ' + url;
54+
msg += '(' + res.statusCode + ') - ';
55+
56+
if (contents && contents.length > 0) {
57+
var jsonObj = null;
58+
try {
59+
jsonObj = JSON.parse(contents);
60+
} catch (e) {}
61+
62+
if (jsonObj && jsonObj.message) {
63+
msg += jsonObj.message;
64+
} else {
65+
msg += url;
66+
}
67+
}
5568

5669
onResult(new Error(msg), res.statusCode, null);
5770
} else {

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "vso-node-api",
33
"description": "Node client for Visual Studio Online/TFS REST APIs",
4-
"version": "0.4.3",
4+
"version": "0.4.4",
55
"main": "./WebApi.js",
66
"repository": {
77
"type": "git",

0 commit comments

Comments
 (0)