我必须创建文件上传,将文件存储在Amazon S3存储桶中,然后将信息写入数据库。
router.post('/report', upload.single('file'), function(req, res, next) {
reportService
.uploadReport(req.body, req.file)
.then(data => {
res.send(data);
})
.catch(err => {
res.send(err);
});
});
这段代码处理呼叫。它会触发以下代码。
function uploadReport(report, file) {
var objectParams = { Bucket: bucketName, Key: file.filename, Body: '' };
var fs = require('fs');
var fileStream = fs.createReadStream(file.path);
fileStream.on('error', function(err) {
console.log('File Error', err);
});
objectParams.Body = fileStream;
return new Promise((resolve, reject) => {
s3.putObject(objectParams)
.promise()
.then(
data => {
resolve(this.addReport(report, data.Location)
.then(response => resolve(response))
.catch(err => {
reject(err);
})
);
},
err => {
reject(err);
}
);
});
}
这段代码将文件上传到我的s3存储桶中,上传完成后,它调用写入数据库的save report function
。
function addReport(report, url) {
return new Promise((resolve, reject) => {
reportModel
.addReport(report, url)
.then(report => {
resolve(report);
})
.catch(err => {
reject(err);
});
});
}
我知道我的2个诺言可以单独工作,但是将它们组合成1个诺言不会触发任何功能。
我如何正确地将它们组合成1个诺言?
答案 0 :(得分:1)
您似乎违反了What is the explicit promise construction antipattern and how do I avoid it?。您正在用承诺解决承诺。相反,您只想返回Promise:
return s3
.putObject(objectParams)
.promise()
.then(data => this.addReport(report, data.location));
答案 1 :(得分:0)
您将必须返回addReport
Promise,然后在then回调中解析/拒绝原始Promise。