【发布时间】:2015-04-01 23:45:48
【问题描述】:
我对节点比较陌生,想编写一个模块,从 S3 存储桶获取图像,调整其大小并将其保存到亚马逊新 Lambda 服务上的临时目录,然后将图像上传回存储桶。
当我运行代码时,似乎没有调用我的任何函数(download、transform 和 upload)。我使用tmp 创建临时目录并使用graphicsMagick 调整图像大小。
我的代码有什么问题?
我已经在模块外部定义了依赖项和数组,因为我还有另一个依赖于这些。
// dependencies
var AWS = require('aws-sdk');
var gm = require('gm').subClass({ imageMagick: true });
var fs = require("fs");
var tmp = require("tmp");
// get reference to S3 client
var s3 = new AWS.S3();
var _800px = {
width: 800,
destinationPath: "large"
};
var _500px = {
width: 500,
destinationPath: "medium"
};
var _200px = {
width: 200,
destinationPath: "small"
};
var _45px = {
width: 45,
destinationPath: "thumbnail"
};
var _sizesArray = [_800px, _500px, _200px, _45px];
var len = _sizesArray.length;
exports.AwsHandler = function(event) {
// Read options from the event.
var srcBucket = event.Records[0].s3.bucket.name;
var srcKey = event.Records[0].s3.object.key;
var dstnKey = srcKey;
// create temporary directory
var tmpobj = tmp.dirSync();
// function to determine paths
function _filePath (directory, i) {
if (!directory) {
return "dst/" + _sizesArray[i].destinationPath + "/" + dstnKey;
} else {
return directory + "/dst/" + _sizesArray[i].destinationPath + "/" + dstnKey;
}
};
// Infer the image type.
var typeMatch = srcKey.match(/\.([^.]*)$/);
if (!typeMatch) {
console.error('unable to infer image type for key ' + srcKey);
return;
};
var imageType = typeMatch[1];
if (imageType != "jpg" && imageType != "png") {
console.log('skipping non-image ' + srcKey);
return;
};
(function resizeImage () {
function download () {
console.log("started!");
s3.getObject({
Bucket: srcBucket,
Key: srcKey
},
function (err, response) {
if (err) {
console.error(err);
}
// call transform if successful
transform (response);
}
);
};
function transform (response) {
for ( var i = 0; i<len; i++ ) {
// define path for image write
var _Key = _filePath (tmpobj, i);
// resize images
gm(response.Body, srcKey)
.resize(_sizesArray[i].width)
.write(_Key, function (err) {
if (err) {
console.error(err);
}
upLoad ();
});
}
};
function upLoad () {
for ( var i = 0; i<len; i++ ) {
var readPath = _filePath (tmpobj, i);
var writePath = _filePath (i);
// read file from temp directory
fs.readFile(readPath, function (err, data) {
if (err) {
console.error(err);
}
// upload images to s3 bucket
s3.putObject({
Bucket: srcBucket,
Key: writePath,
Body: data,
ContentType: data.type
},
function (err) {
if (err) {
console.error(err);
}
console.log("Uploaded with success!");
});
})
}
// Manual cleanup of temporary directory
tmpobj.removeCallback();
};
}());
};
【问题讨论】:
-
你从来没有打电话给
download() -
我的印象是 Lambda 会调用它。但是我刚刚发现我需要配置它必须调用的函数。这是否意味着我的节点代码编写正确?
-
这有点复杂,你可以让它更干净,但我没有发现任何明显的错误。 lambda 将运行,但它所做的只是在内部定义三个函数:下载、转换、上传。然后它调用 tmpObj.removeCallback()。
-
仔细观察你会遇到一个问题:你循环遍历大小,为每个大小调用一次 gm(),然后在 gm 的回调中调用 upLoad()。因此,每当 gm 完成时,upLoad 回调将被调用四次。并且其他格式化的 gm 输出文件可能不可用。
-
如果我在 for 循环之外调用
upLoad会怎样?这不会缓解这个问题吗?
标签: node.js amazon-web-services amazon-s3