Finally fully async.

Author: SaltwaterC

CHANGELOG.md

@@ -20,6 +20,7 @@
  * Implements new GET response handlers: buffer - the response contains a buffer key with the buffer contents; stream - returns the HTTPS response itself which implements node.js's Readable Stream interface.
  * Adds a new s3.putStream() helper for PUT'ing streams to S3.
  * Adds a new s3.putBuffer() helper for PUT'ing buffers to S3.
+ * Goes fully async by removing the only blocking call, fs.statSync(), when using PUT with file request body handler.
  * Unit testing coverage.
 
 ## v0.4.4

lib/aws.js

@@ -236,8 +236,6 @@ var client = function (config) {
 			}
 			if (body.file) {
 				body.file = p.resolve(body.file);
-				// XXX wrap into a callback in order to remove this blocking call
-				headers['content-length'] = fs.statSync(body.file).size;
 			}
 			if ( ! headers['content-length']) {
 				headers = tools.merge(headers, {'content-length': 0});

lib/internals.js

@@ -374,13 +374,20 @@ var standardHeaders = function (config, method, headers, path, body, callback) {
 	hdr = normalizeHeaders(hdr);
 	if (body.file) {
 		if ( ! hdr['content-type']) {
-			mime.fileWrapper(p.resolve(body.file), function (err, res) {
+			mime.fileWrapper(body.file, function (err, res) {
 				if (err) {
 					callback(err);
 				} else {
 					hdr['content-type'] = res;
-					hdr.authorization = authorize(config, method, hdr, path);
-					callback(null, hdr);
+					fs.stat(body.file, function (err, stats) {
+						if (err) {
+							callback(err);
+						} else {
+							hdr['content-length'] = stats.size;
+							hdr.authorization = authorize(config, method, hdr, path);
+							callback(null, hdr);
+						}
+					});
 				}
 			});
 			return;