From 3c0bbc53efa3786dff33fedbb0d03c3eb7e92df1 Mon Sep 17 00:00:00 2001
From: Benjy Weinberger <benjyw@gmail.com>
Date: Tue, 8 Aug 2023 07:24:30 -0500
Subject: [PATCH] Support reading the subject from a file. (#173)

Allows dynamic subjects to be generated at action runtime
(e.g., "Foobar version 1.2.3 is released", where 1.2.3
is not known at action config time).

Leverages the existing file:// support for body, but renames
getBody to getText, in keeping with this expanded role.
---
 .github/workflows/test.yml |  2 +-
 main.js                    | 22 +++++++++++-----------
 testdata/subject.txt       |  1 +
 3 files changed, 13 insertions(+), 12 deletions(-)
 create mode 100644 testdata/subject.txt

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index 3d062d07d..76a8b5195 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -30,7 +30,7 @@ jobs:
               <p>Paragraph</p>
               </body>
               </html>
-          - subject: Plain body (Markdown)
+          - subject: file://testdata/subject.txt
             convert_markdown: true
             body: file://README.md
           - subject: HTML body (Markdown)
diff --git a/main.js b/main.js
index 999c6ccf2..549e1d1f9 100644
--- a/main.js
+++ b/main.js
@@ -5,22 +5,22 @@ const fs = require("fs")
 const showdown = require("showdown")
 const path = require("path")
 
-function getBody(bodyOrFile, convertMarkdown) {
-    let body = bodyOrFile
+function getText(textOrFile, convertMarkdown) {
+    let text = textOrFile
 
-    // Read body from file
-    if (bodyOrFile.startsWith("file://")) {
-        const file = bodyOrFile.replace("file://", "")
-        body = fs.readFileSync(file, "utf8")
+    // Read text from file
+    if (textOrFile.startsWith("file://")) {
+        const file = textOrFile.replace("file://", "")
+        text = fs.readFileSync(file, "utf8")
     }
 
     // Convert Markdown to HTML
     if (convertMarkdown) {
         const converter = new showdown.Converter()
-        body = converter.makeHtml(body)
+        text = converter.makeHtml(text)
     }
 
-    return body
+    return text
 }
 
 function getFrom(from, username) {
@@ -116,14 +116,14 @@ async function main() {
         const info = await transport.sendMail({
             from: getFrom(from, username),
             to: to,
-            subject: subject,
+            subject: getText(subject, false),
             cc: cc ? cc : undefined,
             bcc: bcc ? bcc : undefined,
             replyTo: replyTo ? replyTo : undefined,
             inReplyTo: inReplyTo ? inReplyTo : undefined,
             references: inReplyTo ? inReplyTo : undefined,
-            text: body ? getBody(body, false) : undefined,
-            html: htmlBody ? getBody(htmlBody, convertMarkdown) : undefined,
+            text: body ? getText(body, false) : undefined,
+            html: htmlBody ? getText(htmlBody, convertMarkdown) : undefined,
             priority: priority ? priority : undefined,
             attachments: attachments ? (await getAttachments(attachments)) : undefined,
         })
diff --git a/testdata/subject.txt b/testdata/subject.txt
new file mode 100644
index 000000000..2e2d09274
--- /dev/null
+++ b/testdata/subject.txt
@@ -0,0 +1 @@
+Plain body (Markdown)