函数错误日志写入的正确方式及crontab下日志有效性咨询

背景 #

需求是将输出同时打印到错误日志文件和控制台，现有两种实现方案：

方案1：函数内部包含错误日志逻辑 #

copy_to_s3() {
    local INPUT_FILE_NAME=$1
    local BUCKET_NAME=$2
    if aws s3 cp "${INPUT_FILE_NAME}" "s3://${BUCKET_NAME}" >error.log 2>&1; then
        echo "Successfully copied the input file ${INPUT_FILE} to s3://${BUCKET_NAME}"
    else
        error=$(cat "error.log")
        # EMAIL this error to the admin
        echo "Something went wrong when copying the input file ${INPUT_FILE} to s3://${BUCKET_NAME}"
        exit 1
    fi

    rm -rf "${INPUT_FILE_NAME}"
}

copy_to_s3 "test.tar.gz" "test-s3-bucket"

方案2：调用函数时添加错误日志逻辑 #

copy_to_s3() {
    local INPUT_FILE_NAME=$1
    local BUCKET_NAME=$2
    if aws s3 cp "${INPUT_FILE_NAME}" "s3://${BUCKET_NAME}"; then
        echo "Successfully copied the input file ${INPUT_FILE} to s3://${BUCKET_NAME}"
    else
        echo "Something went wrong when copying the input file ${INPUT_FILE} to s3://${BUCKET_NAME}"
        exit 1
    fi

    rm -rf "${INPUT_FILE_NAME}"
}

copy_to_s3 "test.tar.gz" "test-s3-bucket" >error.log 2>&1

问题与解答 #

1. 上述两种方法中哪种更推荐？ #

推荐方案2，核心原因是遵循「单一职责」原则，具体优势：

• 函数职责纯粹：copy_to_s3只负责完成S3文件复制的核心逻辑，日志记录属于调用场景的附加需求，交给外部处理能让函数更通用，后续在不需要日志的场景调用时无需修改函数代码。
• 灵活性更强：可以根据不同调用场景自由指定日志路径、日志模式（比如追加>>而非覆盖>），甚至更换日志处理方式；而方案1的日志路径硬编码在函数内部，修改成本高，无法适配多场景。
• 降低耦合风险：函数内部不依赖日志文件的写入权限，避免了因日志目录权限问题导致核心复制逻辑执行失败的额外问题。

另外注意：两种方案的echo语句中都存在变量错误——函数内定义的是INPUT_FILE_NAME，但输出时误用了未定义的${INPUT_FILE}，会导致输出内容缺失文件名，需要修正为${INPUT_FILE_NAME}。

2. 若将该脚本放入如下crontab配置中，是否仍能记录错误？ #

crontab -u ec2-user - <<EOF
PATH=/usr/local/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/home/ec2-user/.local/bin:/home/ec2-user/bin
0 0,4,8,12,16,20 * * * /home/ec2-user/test.sh
EOF

能正常记录错误，但需要注意日志文件的生成路径：

• 对于方案1：函数内部已将aws s3 cp的输出重定向到error.log，cron执行脚本时的工作目录是ec2-user的主目录（/home/ec2-user），因此error.log会生成在该目录下，只要目录有写入权限就能正常记录aws命令的错误输出。不过函数中的echo语句会输出到cron的标准输出，默认情况下cron会将这部分内容通过邮件发送给ec2-user（需系统邮件配置正常）。
• 对于方案2：脚本调用函数时的>error.log 2>&1会将函数的所有标准输出和错误输出都写入/home/ec2-user/error.log，所有日志会集中存储在这里，不会触发cron的邮件通知（除非存在未被重定向的输出）。

此外crontab中已指定完整的PATH环境变量，确保aws命令能被正确找到，不会因环境变量缺失导致命令执行失败。

内容的提问来源于stack exchange，提问作者PirateApp