feat: use fast-glob instead of globby (#285)

We have been using an outdated version of globby because the new version is not compatible. This poses as risk as we are no longer receiving security updates.
google-github-actions · Jun 21, 2022 · c68f13b · c68f13b
1 parent d8dc20b
commit c68f13b
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 14 deletions.
diff --git a/dist/index.js b/dist/index.js
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -26,7 +26,7 @@
     "@actions/core": "^1.9.0",
     "@google-cloud/storage": "^6.1.0",
     "@google-github-actions/actions-utils": "^0.4.0",
-    "globby": "^11.0.4",
+    "fast-glob": "^3.2.1",
     "ignore": "^5.2.0"
   },
   "devDependencies": {

diff --git a/src/util.ts b/src/util.ts
@@ -18,7 +18,7 @@ import { promises as fs } from 'fs';
 import * as path from 'path';
 import * as v8 from 'v8';
 
-import globby from 'globby';
+import fg from 'fast-glob';
 import { toPlatformPath, toPosixPath } from '@google-github-actions/actions-utils';
 
 /**
@@ -97,8 +97,10 @@ export function parseBucketNameAndPrefix(name: string): [bucket: string, prefix:
  */
 export async function expandGlob(directoryPath: string, glob: string): Promise<string[]> {
   const directoryPosix = toPosixPath(directoryPath);
-  const pth = toPosixPath(path.posix.join(directoryPosix, glob));
-  const filesList = await globby([pth], {
+  const search = toPosixPath(glob || '**/*');
+  const filesList = await fg(search, {
+    absolute: true,
+    cwd: directoryPath,
     dot: true,
   });