我在使用一些我正在使用AWS Lambda的node.js代码时遇到了一些问题。我有一些我需要做的异步调用,虽然第一个表现得像我期望的那样,lambda函数在第二个调用完成之前终止。
返回为null,这让我觉得lambda正在触及它的隐式回调,但是我认为它不应该这样做,而有一个尚未解决的承诺。
代码:
exports.handle = async function(event, context) {
var AWS = require("aws-sdk");
AWS.config.update({
region: "eu-west-1",
endpoint: "dynamodb.eu-west-1.amazonaws.com"
});
var docClient = new AWS.DynamoDB.DocumentClient();
console.log("Scanning Catalogue...");
var params = {
TableName : "Catalogue"
};
await docClient.scan(params).promise().then(function (data) {
console.log("Scan succeeded.");
data.Items.forEach(function (item) {
//console.log(item.url);
checkIfUpdateRequired(item);
})
})
}
async function checkIfUpdateRequired (catalogueItem) {
var request = require("request-promise");
console.log("Getting " + catalogueItem.url);
await request(catalogueItem.url).then(function(response) {
console.log(response.statusCode);
console.log(response.headers['content-type']);
});
}
日志输出:
START RequestId: 634a55b7-6258-11e8-9f18-6b300c3b5de1 Version: $LATEST
2018-05-28T09:20:44.425Z 634a55b7-6258-11e8-9f18-6b300c3b5de1 Scanning Catalogue...
2018-05-28T09:20:45.446Z 634a55b7-6258-11e8-9f18-6b300c3b5de1 Scan succeeded.
2018-05-28T09:20:47.967Z 634a55b7-6258-11e8-9f18-6b300c3b5de1 Getting https://raw.githubusercontent.com/BSData/wh40k/master/Aeldari%20-%20Craftworlds.cat
2018-05-28T09:20:48.028Z 634a55b7-6258-11e8-9f18-6b300c3b5de1 Getting https://raw.githubusercontent.com/BSData/wh40k/master/Imperium%20-%20Adeptus%20Custodes.cat
END RequestId: 634a55b7-6258-11e8-9f18-6b300c3b5de1
REPORT RequestId: 634a55b7-6258-11e8-9f18-6b300c3b5de1 Duration: 7882.68 ms Billed Duration: 7900 ms Memory Size: 128 MB Max Memory Used: 49 MB
因此日志告诉我调用了checkIfUpdateRequired(),但是在履行承诺之前,lambda结束(报告成功,结果值为null)。我没有对处理程序进行任何手动返回或回调,这似乎是与lambda结束'早期'相关的问题的标准。
我的智慧结束了,有人可以提出任何建议吗?
答案 0 :(得分:3)
您不等待checkIfUpdateRequired
承诺完成; docClient.scan
中的所有内容在原始代码中都是同步的。使用Promise.all
等待所有承诺完成:
await docClient.scan(params).promise().then(function (data) {
console.log("Scan succeeded.");
return Promise.all(data.Items.map(checkIfUpdateRequired));
});
请注意,如果您正在使用await
,那么如果您习惯性地使用代码而不是.then
,那么您的代码将更加平坦且易于理解。例如,您可以重构为:
const data = await docClient.scan(params).promise();
return Promise.all(data.Items.map(checkIfUpdateRequired));
和
async function checkIfUpdateRequired (catalogueItem) {
// actually, better to only require once, rather than on every function call
const request = require("request-promise");
console.log("Getting " + catalogueItem.url);
const response = await request(catalogueItem.url)
console.log(response.statusCode);
console.log(response.headers['content-type']);
}