fix: support symlinks when commit signing (#3359)

peter-evans · web-flow · commit 5e914681df9d · 2024-09-18T10:40:10.000-07:00
diff --git a/dist/index.js b/dist/index.js
@@ -1781,7 +1781,13 @@ function readFile(path) {
     return fs.readFileSync(path, 'utf-8');
 }
 function readFileBase64(pathParts) {
-    return fs.readFileSync(path.resolve(...pathParts)).toString('base64');
+    const resolvedPath = path.resolve(...pathParts);
+    if (fs.lstatSync(resolvedPath).isSymbolicLink()) {
+        return fs
+            .readlinkSync(resolvedPath, { encoding: 'buffer' })
+            .toString('base64');
+    }
+    return fs.readFileSync(resolvedPath).toString('base64');
 }
 /* eslint-disable  @typescript-eslint/no-explicit-any */
 function hasErrorCode(error) {
diff --git a/src/utils.ts b/src/utils.ts
@@ -127,7 +127,13 @@ export function readFile(path: string): string {
 }
 
 export function readFileBase64(pathParts: string[]): string {
-  return fs.readFileSync(path.resolve(...pathParts)).toString('base64')
+  const resolvedPath = path.resolve(...pathParts)
+  if (fs.lstatSync(resolvedPath).isSymbolicLink()) {
+    return fs
+      .readlinkSync(resolvedPath, {encoding: 'buffer'})
+      .toString('base64')
+  }
+  return fs.readFileSync(resolvedPath).toString('base64')
 }
 
 /* eslint-disable  @typescript-eslint/no-explicit-any */

Original file line number	Diff line number	Diff line change
`@@ -127,7 +127,13 @@ export function readFile(path: string): string {`
`127`	`127`	`}`
`128`	`128`
`129`	`129`	`export function readFileBase64(pathParts: string[]): string {`
`130`		`- return fs.readFileSync(path.resolve(...pathParts)).toString('base64')`
	`130`	`+ const resolvedPath = path.resolve(...pathParts)`
	`131`	`+ if (fs.lstatSync(resolvedPath).isSymbolicLink()) {`
	`132`	`+ return fs`
	`133`	`+ .readlinkSync(resolvedPath, {encoding: 'buffer'})`
	`134`	`+ .toString('base64')`
	`135`	`+ }`
	`136`	`+ return fs.readFileSync(resolvedPath).toString('base64')`
`131`	`137`	`}`
`132`	`138`
`133`	`139`	`/* eslint-disable @typescript-eslint/no-explicit-any */`