easier fix for Content-Length issue.

johnnycrab · Jun 12, 2015 · a7b61d7 · johnnycrab · Jun 12, 2015 · a7b61d7
1 parent 24ac455
commit a7b61d7
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/lib/vimeo.js b/lib/vimeo.js
@@ -86,7 +86,6 @@ Vimeo.prototype.access_token = null;
 Vimeo.prototype.request = function vimeo_request (options, callback) {
 	var client = null;
 	var writeBody = null;
-	var needsContentLength = false;
 
 	// if a url was provided, build an options object
 	if (typeof options === "string") {
@@ -104,7 +103,6 @@ Vimeo.prototype.request = function vimeo_request (options, callback) {
 
 	// Write the request body
 	if (['POST','PATCH','PUT','DELETE'].indexOf(request_options.method) !== -1) {
-		needsContentLength = true;
 		if (request_options.headers['Content-Type'] === 'application/json') {
 			writeBody = JSON.stringify(options.query);
 		} else {
@@ -113,7 +111,7 @@ Vimeo.prototype.request = function vimeo_request (options, callback) {
 
 	}
 
-	if (needsContentLength) request_options.headers['Content-Length'] = writeBody ? Buffer.byteLength(writeBody, 'utf8') : 0;
+	if (writeBody) request_options.headers['Content-Length'] = Buffer.byteLength(writeBody, 'utf8');
 
 	// Locate the proper client from the request protocol
 	client = request_options.protocol === 'https:' ? https_module : http_module;
@@ -192,6 +190,7 @@ Vimeo.prototype._buildRequestOptions = function (options) {
 	// Set proper headers for POST, PATCH and PUT bodies
 	if (['POST','PATCH','PUT','DELETE'].indexOf(request_options.method) !== -1 && !request_options.headers['Content-Type']) {
 		request_options.headers['Content-Type'] = 'application/json';
+		request_options.headers['Content-Length'] = 0;
 
 	// Apply parameters to the url for GET requests
 	} else if (request_options.method === 'GET') {