mirrors/axios
2
0
Fork 0
mirror of https://github.com/tenrok/axios.git synced 2026-06-08 17:22:34 +03:00
Code Activity

Handle UTF-8 multibyte sequences in node

Browse Source 
Content-length header was set to the length of the string, which
does not take into account multibyte sequences in UTF-8 strings.

Now converts first to a buffer to get the proper length.
This commit is contained in:
Mathieu Bruyen
2014-09-21 17:48:26 +02:00
parent 095a204c5b
commit 9d5a8781e3
1 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/adapters/http.js
+8 -5
View File
@@ -14,10 +14,6 @@ module.exports = function httpAdapter(resolve, reject, config) {
config.transformRequest
);
if (utils.isArrayBuffer(data)) {
data = new Buffer(new Uint8Array(data));
}
// Merge headers
var headers = utils.merge(
defaults.headers.common,
@@ -25,8 +21,15 @@ module.exports = function httpAdapter(resolve, reject, config) {
config.headers || {}
);
// Add Content-Length header if data exists
if (data) {
if (utils.isArrayBuffer(data)) {
data = new Buffer(new Uint8Array(data));
} else if (utils.isString(data)) {
data = new Buffer(data, 'utf-8');
} else {
return reject(new Error('Data after transformation must be a string or an ArrayBuffer'));
}
// Add Content-Length header if data exists
headers['Content-Length'] = data.length;
}