coolaj86
/
request.js
mirror of https://github.com/therootcompany/request.js.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

bugfix should set application/json on json post unless content-type already set

This commit is contained in:
AJ ONeal 2018-06-20 02:11:37 -06:00
parent bdf5262f1b
commit c9e8c4702b
1 changed files with 28 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
index.js
View File

@ -29,6 +29,14 @@ function mergeOrDelete(defaults, updates) {
return updates;
}
function hasHeader(reqOpts, header) {
var headers = {};
Object.keys(reqOpts.headers).forEach(function (key) {
headers[key.toLowerCase()] = true;
});
return headers[header.toLowerCase()];
}
function toJSONifier(keys) {
return function () {
@ -186,7 +194,7 @@ function setDefaults(defs) {
});
if (_body) {
debug("\n[urequest] body");
debug("\n[urequest] '" + finalOpts.method + "' (request) body");
debug(_body);
// used for chunked encoding
//req.write(_body);
@ -209,6 +217,16 @@ function setDefaults(defs) {
if ('string' === typeof opts) {
opts = { url: opts };
}
module.exports._keys.forEach(function (key) {
if (key in opts && 'undefined' !== typeof opts[key]) {
reqOpts[key] = opts[key];
} else if (key in defs) {
reqOpts[key] = defs[key];
}
});
// TODO url.resolve(defs.baseUrl, opts.url);
if ('string' === typeof opts.url || 'string' === typeof opts.uri) {
if ('string' === typeof opts.url) {
reqOpts.url = opts.url;
@ -228,19 +246,17 @@ function setDefaults(defs) {
//reqOpts.uri = url.parse(reqOpts.url);
}
}
reqOpts.method = (opts.method || 'GET').toUpperCase();
reqOpts.headers = opts.headers || {};
if ((true === reqOpts.json && reqOpts.body) || reqOpts.json) {
reqOpts.headers['Content-Type'] = 'application/json';
}
module.exports._keys.forEach(function (key) {
if (key in opts && 'undefined' !== typeof opts[key]) {
reqOpts[key] = opts[key];
} else if (key in defs) {
reqOpts[key] = defs[key];
reqOpts.method = (reqOpts.method || 'GET').toUpperCase();
reqOpts.headers = reqOpts.headers || {};
// crazy case for easier testing
if (!hasHeader(reqOpts, 'CoNTeNT-TyPe')) {
if ((true === reqOpts.json && reqOpts.body)
|| (true !== reqOpts.json && reqOpts.json)) {
reqOpts.headers['Content-Type'] = 'application/json';
}
});
}
return urequestHelper(reqOpts, cb);
}