AWS Lambda SNS 两次发送主题

AWS Lambda SNS sending topic twice

TL;DR

编写一个 lambda 函数来执行一些数据库查询,然后向某些用户组发送电子邮件。

会员收到邮件两次。

该应用是一款音乐流媒体应用,用户可以在其中创作歌曲。他们还可以创建群组,邀请成员加入这些群组,并将他们的歌曲分享到这些群组。

这是通过 API 调用的 lambda:

  const shareWithGroup = async event => {
  const { songCuid, groupCuids } = JSON.parse(event.body);
  const shareSongDB = await query(
    sql.queryShareWithGroup(songCuid, groupCuids),
  ); //share to group in DB
  if (!shareSongDB) {
    return corsUtil.failureWithCors("Couldn't Share Song with group");
  }
  const song = await query(sql.queryRead(songCuid));
  if (!song) {
    return corsUtil.failureWithCors('Song doesnt exist');
  }
  const songTitle = song.rows[0].songTitle; //retrieve songTitle
  const promises = groupCuids.map(async groupCuid => {
    console.log('GROUP_CUID', groupCuid);
    const emailResults = await query(sql.queryReadGroupEmails(groupCuid)); // get emails for group + groupName
    const results = emailResults.rows;
    const groupName = results[0].groupName;
    let emails = [];
    results.map(row => {
      emails.push(row.email); //push email address into array
    });
    const payload = JSON.stringify({ groupName, emails, songTitle }); //send groupName, emails list and songTitle to SNS to trigger email
    console.log(payload);
    await publishSNS(payload)
    //send the topic
  });

  //Resolve all promises
  await Promise.all(promises);
  console.log(promises);
  return corsUtil.successWithCors('Success');
};

const publishSNS = async payload => {
  console.log('publishing sns topic');
  //SEND EMAILS
  const params = {
    Message: payload,
    TopicArn: `arn:aws:sns:eu-west-1:${process.env.AWS_ACC_ID}:${process.env.STAGE}-songShareTrigger`,
  };

  return await sns
    .publish(params, async error => {
      if (error) {
        console.error(error);
        //TODO: Actually fail the function - can't do with lambdaFactory
      }
    })
    .promise();
};

和一个示例请求(将歌曲分享给一组):

{"songCuid":"XXX","groupCuids":["XXX"]}

问题是,尽管它应该只发送 1 个 SNS 主题,但它却发送了 2 个。当我将一首歌分享给 2 个群组时,它发送了 4 个 SNS 主题。

这是由 SNS 主题触发的 lambda:

const aws = require('aws-sdk');
const ses = new aws.SES();
const corsUtil = require('../utils/corsUtil');

exports.songShareEmail = (event, context) => {
  console.log('EVENT : ', event.Records[0].Sns); // LOG SNS
  const body = JSON.parse(event.Records[0].Sns.Message);
  const { groupName, emails, songTitle } = body;
  console.log('Body : ', body);

  const groupInviteEmailData = `
      <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" 
      "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
      <html>
      <head>
        <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
        <meta name="viewport" content="width=device-width, initial-scale=1.0"/>
      </head>
      <body>
        <div class="email-body" style="max-width:900px; margin:auto;" >
          <div class="content" style="background-color:white; margin:0 auto; 
            display:block;">
            <p>Hello<p>
            </br>
            A new song has been shared to the group : <u>${groupName}</u><br>
            <h3>Song Title: ${songTitle}</h3>

          </div>
        </div>
      </body>
      </html>
  `;

    var params = {
        Destination: {
          ToAddresses: emails,
        },
        Message: {
          Body: {
            Html: {
              Charset: 'UTF-8',
              Data: groupInviteEmailData,
            },
          },
          Subject: {
            Charset: 'UTF-8',
            Data: `New song shared to ${groupName}`,
          },
        },
        Source: 'xxx',
      };
      console.log('sendingEmail');
      ses.sendEmail(params, function(err) {
        if (err) {
          console.log(err);
          const response = corsUtil.failureWithCors(err);
          context.fail(response);
        } else {
          context.succeed('Done');
        }
      });
    };

我已经记录了事件中的 SNS 对象,我可以看到它每次都是一个新的 MessageId,这意味着它实际上每次发送 2 个主题,而不是 lambda 被触发两次同一个SNS。

有什么办法解决这个问题吗?

我知道这个问题很老了,但我发现自己也遇到了同样的问题。我解决了我的错误,我相信你也有同样的问题。

你的publishSNS方法中的.promise()不需要,直接去掉(下面注释掉)。

const publishSNS = async payload => {
  console.log('publishing sns topic');
  //SEND EMAILS
  const params = {
    Message: payload,
    TopicArn: `arn:aws:sns:eu-west-1:${process.env.AWS_ACC_ID}:${process.env.STAGE}-songShareTrigger`,
  };

  return await sns
    .publish(params, async error => {
      if (error) {
        console.error(error);
        //TODO: Actually fail the function - can't do with lambdaFactory
      }
    });
    //.promise(); 
};

我对 iotData 有相同的 pb,我使用类似的东西来解决我的问题

const request = iotdata.publish(mqttParams);
request
    .on('success', () => console.log("Success"))
    .on('error', () => console.log("Error"))
return new Promise(() => request.send());

所选答案只对了一半。

我也遇到了同样的问题。您需要将 SNS 与回调一起使用,或者使用 SNS 在构造函数中设置 api 版本并使用 promise()

const sns = new SNS({ apiVersion: '2010-03-31' });
await sns
  .publish(
    {
      Message: 'Message',
      TopicArn: <Arn>,
    },
  )
  .promise();

如果您不在构造函数中传递 api版本,则必须传递回调并且不使用 promise()