使用Bash脚本的邮件输出

Question

使用生成的SSH密钥从主机A到几台主机（现在仅在下面列出的一台主机）进行SSH，然后转到特定文件，grep表示日期为昨天的特定单词..然后我想将此输出通过电子邮件发送给我。

它正在发送电子邮件，但给了我命令，而不是命令的输出。

#!/bin/bash

HOST="XXXXXXXXXXXXXXXXXX, XXXXXXXXXXXXX"

DATE=$(date -d "yesterday")

INVALID=' cat /xxx/xxx/xxxxx | grep 'WORD' | sed 's/$/.\n/g' | grep "$DATE"'

COUNT=$(echo "$INVALID" | wc -c)

for x in $HOSTS

do
ssh BLA@"$x" $COUNT

if [ "$COUNT" -gt 1 ];
then

    EMAILTEXT=""
        if [ "$COUNT" -gt 1 ];
        then
                EMAILTEXT="$INVALID"
        fi
fi

done | echo -e "$EMAILTEXT" | mail XXXXXXXXXXX.com

Answer 1

这不是试图回答您的问题的正确方法，但是我认为您应该意识到代码的一些基本问题。

INVALID=' cat /xxx/xxx/xxxxx | grep 'WORD' | sed 's/$/.\n/g' | grep "$DATE"'

这为变量INVALID分配了一个简单的字符串。 由于存在引用问题，因此根本不会引用s/$/.\\n/g ，并且可能会被shell破坏。 （您不能嵌套单引号-第一个单引号的字符串从第一个引号延伸到下一个引号，然后WORD在任何引号之外，然后是下一个单引号字符串，依此类推。）

如果您打算在此时将其作为命令执行，则您正在寻找命令替代品。 剥夺了无用的多层性，也许像

INVALID=$(sed -n -e '/WORD/!d' -e "/$DATE/s/$/./p" /xxx/xxx/xxxx)

它会寻找与WORD和$DATE匹配的行，并在匹配项的末尾附加一个点-我认为这就是您的代码的精髓所在，但是如果不进一步了解此代码应该做什么，就无法分辨如果这是您真正需要的。

COUNT=$(echo "$INVALID" | wc -c)

这将为$COUNT分配一个数字。 使用INVALID的静态定义，该数字始终为62； 但我想这实际上不是您想要的。

for x in $HOSTS    
do
  ssh BLA@"$x" $COUNT

这试图在多个远程主机上作为命令执行该编号（除非循环通过HOSTS并且包含主机的变量仅命名为HOST ）。 除非您拥有一系列称为自然数的命令，这些命令在这些远程主机上做有用的事情，否则这可能不会有用。 但是我认为可以肯定地说，这不是这里应该发生的事情（如果确实如此，则绝对有必要在您的问题中对此进行解释）。

  if [ "$COUNT" -gt 1 ];
  then

    EMAILTEXT=""
        if [ "$COUNT" -gt 1 ];
        then
                EMAILTEXT="$INVALID"
        fi
  fi

因此， EMAILTEXT是空字符串或INVALID的值。 您将其分配为上面的静态字符串，这可能是您的直接问题的根源。 但是，即使以某种方式将其分配给本地主机上的命令，您为什么还需要访问远程主机并在那里执行某些操作？ 还是您实际上打算在每个远程主机上执行命令并获取输出？

done | echo -e "$EMAILTEXT" | mail XXXXXXXXXXX.com

用管道echo根本没有意义，因为它不读取其标准输入。 你或许应该只是后一个换行符done ; 尽管可能更有用的安排是让您的循环产生输出，然后我们通过管道发送到mail 。

纯粹是推测性的，也许您真正想要的是以下内容。

for host in $HOSTS; do
    ssh BLA@"$host" sed -n -e '/WORD/!d' -e "/$DATE/s/$/./p" /xxx/xxx/xxxx |
    grep . || echo INVALID
done | mail XXXXXXXXXXX.com

如果要检查是否存在多于一行的输出（这是-gt 1建议），则可能需要稍微复杂一点。

Answer 2

您的命令替换无效。 您应该阅读它的工作原理，但这是问题所在：

COUNT=$(echo "$INVALID" | wc -c)
[...]
ssh BLA@"$x" $COUNT

应该：

COUNT_CMD="'${INVALID} | wc -c'"
[...]
COUNT=$(ssh BLA@"$x" $COUNT_CMD)

这会将$INVALID的值插入字符串，并将整个内容放在单引号中。 ssh调用必须使用单引号，因此不会在脚本中而是在远程主机上评估管道。 （出于可读性/清晰度，将COUNT更改为COUNT_CMD 。）

编辑：

我看错了问题，并且更正了我的答案。