Skip to content
Browse files

Merge pull request #1 from tapvt/master

`commonMethod()` now returns in same form as `request()`
WARNING: This will break existing code, but it seems better to do it now rather than later.  As far as I know, we have no other users.
  • Loading branch information...
2 parents ecb83ed + 1e4d39b commit 7c26749aa6a7966e68964b2134cb1aaa3587eecc @dbrand666 committed Mar 17, 2013
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/discovery-services-client.js
4 lib/discovery-services-client.js
@@ -165,7 +165,7 @@ function processMethods(initOpts, baseUrl, methods, resourceEntry, cb) {
This is a jslint-approved version. The downside is that all the
generated functions are anonymous - any stack traces will be a
bit harder to chase down.
resourceEntry[name] = async.apply(commonMethod, {
apiKey: initOpts.apiKey,
baseUrl: baseUrl,
@@ -221,7 +221,7 @@ function commonMethod(data, options, cb) {
uri: uri,
json: true
}, function (err, res, body) {
- cb(err, body);
+ cb(err, res, body);

0 comments on commit 7c26749

Please sign in to comment.
Something went wrong with that request. Please try again.