ensure subject globs match only files (#54)

Signed-off-by: Brian DeHamer <bdehamer@github.com>
actions · May 6, 2024 · d442d85 · d442d85
1 parent c58d52c
commit d442d85
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 0 deletions.
diff --git a/__tests__/subject.test.ts b/__tests__/subject.test.ts
@@ -203,6 +203,19 @@ describe('subjectFromInputs', () => {
       })
     })
 
+    describe('when a file glob is supplied which also matches non-files', () => {
+      beforeEach(async () => {
+        process.env['INPUT_SUBJECT-PATH'] = `${dir}*`
+      })
+
+      it('returns the subjects (excluding non-files)', async () => {
+        const subjects = await subjectFromInputs()
+
+        expect(subjects).toBeDefined()
+        expect(subjects).toHaveLength(7)
+      })
+    })
+
     describe('when a comma-separated list is supplied', () => {
       beforeEach(async () => {
         process.env['INPUT_SUBJECT-PATH'] =

diff --git a/dist/index.js b/dist/index.js
diff --git a/src/subject.ts b/src/subject.ts
@@ -56,6 +56,11 @@ const getSubjectFromPath = async (
     const files = await glob.create(subPath).then(async g => g.glob())
 
     for (const file of files) {
+      // Skip anything that is NOT a file
+      if (!fs.statSync(file).isFile()) {
+        continue
+      }
+
       const name = subjectName || path.parse(file).base
       const digest = await digestFile(DIGEST_ALGORITHM, file)