Question

我有一个由发布/订阅事件触发的Cloud Function。我使用sendgrid nodejs api。主要想法是向我的客户发送每周统计信息电子邮件。每个客户端运行sendEmail（）函数（80次）。但是，当我检查功能日志时，我看到成功发送了25-30封客户电子邮件，但其余邮件却显示错误：“套接字挂断”

我缩短了整个代码，以显示与发送电子邮件有关的主要部分。这是最后一部分。

    // I shortened the whole function as it is a very long function.
    // The main and the last part is as below
    // I have nearly 80 clients and sendEmail function run for each client.

    function calcData(i, data) {
        return admin.database().ref('clientUrlClicks/' + data.key)
            .orderByChild('date')
            .startAt(dateStartEpox)
            .endAt(dateEndEpox)
            .once('value', urlClickSnap => {
                clients[i].clickTotalWeek = urlClickSnap.numChildren();
                clients[i].listTotalWeek = 0;
                admin.database().ref('clientImpressions/' + data.key)
                    .orderByKey()
                    .startAt(dateStart)
                    .endAt(dateEnd)
                    .once('value', snap => {
                        snap.forEach(function(impressionSnap) {
                            clients[i].listTotalWeek += impressionSnap.val();
                        })
                    }).then(resp => {
                        return sendEmail(i, clients[i]);
                    }).catch(err => {
                        console.log(err);
                    });
            }).catch(err => {
                clients[i].clickTotalWeek = 0;
                console.log(err);
            });
    }

   function sendEmail(i, data) {
        var options = {
            method: 'POST',
            url: 'https://api.sendgrid.com/v3/mail/send',
            headers:
            {
                'content-type': 'application/json',
                authorization: 'Bearer ' + sgApiKey
            },
            body:
            {
                personalizations:
                    [{
                        to: [{ email: data.email, name: data.name }],
                        dynamic_template_data:
                        {
                            dateStart: xxx,
                            dateEnd: xxx,
                        }
                    }],
                from: { email: 'info@xxx.com', name: 'xxx' },
                reply_to: { email: 'info@xxx.com', name: 'xxx' },
                template_id: 'd-f44eeexxxxxxxxxxxxx'
            },
            json: true
        };

        request(options, function (error, response, body) {
            if (error) {
                console.log("err: " + error);
                return;
            }
            return;
        });
    }

编辑：

除了以下与“正确地链接承诺”有关的答案外，我还将所有电子邮件和个性化设置添加到“个性化”数组中，作为“ sendEmail”功能上的对象。因此，我对每个电子邮件都发出了一个请求。现在没问题。

Answer 1

您没有正确链接承诺，因此没有在链接结束时返回最终承诺，这对于Cloud Function是必需的。

以下一组修改是解决此问题的首次尝试。

此外，还不清楚如何调用Sendgrid并返回Sendgrid调用返回的Promise。我建议您使用send()方法，该方法返回Promise，如Node.js的Sendgrid v3 Web API的文档中所述，请参见https://github.com/sendgrid/sendgrid-nodejs/tree/master/packages/mail。

function calcData(i, data) {
        //Declare clients aray here
        return admin.database().ref('clientUrlClicks/' + data.key)
            .orderByChild('date')
            .startAt(dateStartEpox)
            .endAt(dateEndEpox)
            .once('value')
            .then(urlClickSnap => {
                clients[i].clickTotalWeek = urlClickSnap.numChildren();
                clients[i].listTotalWeek = 0;
                return admin.database().ref('clientImpressions/' + data.key)  //Here you didn't return the promise
                    .orderByKey()
                    .startAt(dateStart)
                    .endAt(dateEnd)
                    .once('value');
             .then(snap => {
                     snap.forEach(function(impressionSnap) {
                         clients[i].listTotalWeek += impressionSnap.val();
                    })
                    return sendEmail(i, clients[i]);
             }).catch(err => {
                clients[i].clickTotalWeek = 0;
                console.log(err);
                return null;
            });
    }

Answer 2

我看到您的代码中有两个与Promise链相关的问题，这可能是导致此问题的原因。

首先，您正在sendEmail函数中使用带回调的请求。这根本不会等待您的网络调用完成并返回该函数。现在，这将并行建立调用，并且在您击中80个客户端之前，您的云功能的执行将完成。解决方案是将request-promise-native（https://github.com/request/request-promise-native）库与您的请求库一起使用。因此，您的sendEmail函数现在将变为

sendEmail (i, data) {
    .
    .
    .
    return rpn(options).then((d)=>{return d}).catch((e)=>{return console.log(e)})
}

其他解决方案是将sendgrid客户端用于nodejs，这将仅返回诺言，而无需使用请求。 https://github.com/sendgrid/sendgrid-nodejs/tree/master/packages/mail

第二个问题是您要求从firebase读取数据，在此您还使用回调而不是promise。正确的解决方案是：

function calcData(i, data) {
        return admin.database().ref('clientUrlClicks/' + data.key)
            .orderByChild('date')
            .startAt(dateStartEpox)
            .endAt(dateEndEpox)
            .once('value').then( urlClickSnap => {
                clients[i].clickTotalWeek = urlClickSnap.numChildren();
                clients[i].listTotalWeek = 0;
                return admin.database().ref('clientImpressions/' + data.key)
                    .orderByKey()
                    .startAt(dateStart)
                    .endAt(dateEnd)
                    .once('value').then( snap => {
                        snap.forEach(function(impressionSnap) {
                            clients[i].listTotalWeek += impressionSnap.val();
                        })
                        return sendEmail(i, clients[i]);
                    })
                    .catch(err => {
                        console.log(err);
                    });
            }).catch(err => {
                clients[i].clickTotalWeek = 0;
                console.log(err);
            });
    }

这将确保在完成所有链接的promise的执行后，您的函数calcData将返回。

如果要在循环中调用calcData，请再做一件事，然后确保将所有的Promise存储在数组中，并在循环调用Promise.all(promisesArray)之后，以便函数等待所有执行完成。 / p>

带有Firebase Cloud Functions的SendGrid错误：“套接字挂断”

2 个答案: