Skip to content

Commit

Permalink
feat(wrapper): log request informations in errors
Browse files Browse the repository at this point in the history
  • Loading branch information
Valetudox authored and jzsfkzm committed Apr 11, 2017
1 parent a254c04 commit c365283
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions wrapper.js
Original file line number Diff line number Diff line change
Expand Up @@ -54,28 +54,28 @@ RequestWrapper.prototype = {

request[method](reqOptions, function(err, response) {
if (err) {
logger.error('fatal_error', err.message);
logger.error('fatal_error', err.message, this._getLogParameters());
return reject(new SuiteRequestError(err.message, 500));
}

if (!this.requestOptions.allowEmptyResponse && !response.body) {
logger.error('server_error', 'empty response data');
logger.error('server_error', 'empty response data', this._getLogParameters());
return reject(new SuiteRequestError('Empty http response', 500, response.statusMessage));
}

if (this._isJsonResponse(response)) {
try {
response.body = JSON.parse(response.body);
} catch (ex) {
logger.error('fatal_error', ex);
logger.error('fatal_error', ex, this._getLogParameters());
return reject(new SuiteRequestError(ex.message, 500));
}
}

if (response.statusCode >= 400) {
logger.error('server_error', response.body.replyText, {
logger.error('server_error', response.body.replyText, this._getLogParameters({
code: response.statusCode
});
}));
return reject(new SuiteRequestError(
'Error in http response (status: ' + response.statusCode + ')',
response.statusCode,
Expand Down

0 comments on commit c365283

Please sign in to comment.