fix: use cTimeMs instead of birthtime due to EFS (#1035)

isomerpages · Nov 14, 2023 · c74ee84 · c74ee84
1 parent 54b44c1
commit c74ee84
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/src/services/db/GitFileSystemService.ts b/src/services/db/GitFileSystemService.ts
@@ -858,7 +858,7 @@ export default class GitFileSystemService {
           mediaUrl: `${dataUrlPrefix},${file.content}`,
           mediaPath: `${directoryName}/${fileName}`,
           type: fileType,
-          addedTime: stats.birthtimeMs,
+          addedTime: stats.ctimeMs,
           size: stats.size,
         })
       })
@@ -929,7 +929,7 @@ export default class GitFileSystemService {
                 sha,
                 path,
                 size: type === "dir" ? 0 : stats.size,
-                addedTime: stats.birthtimeMs,
+                addedTime: stats.ctimeMs,
               }
 
               return okAsync(result)

diff --git a/src/services/db/__tests__/GitFileSystemService.spec.ts b/src/services/db/__tests__/GitFileSystemService.spec.ts
@@ -91,7 +91,7 @@ describe("GitFileSystemService", () => {
         path: "fake-dir",
         size: 0,
         addedTime: fs.statSync(`${EFS_VOL_PATH_STAGING}/fake-repo/fake-dir`)
-          .birthtimeMs,
+          .ctimeMs,
       }
       const expectedAnotherFakeDir: GitDirectoryItem = {
         name: "another-fake-dir",
@@ -101,7 +101,7 @@ describe("GitFileSystemService", () => {
         size: 0,
         addedTime: fs.statSync(
           `${EFS_VOL_PATH_STAGING}/fake-repo/another-fake-dir`
-        ).birthtimeMs,
+        ).ctimeMs,
       }
       const expectedFakeEmptyDir: GitDirectoryItem = {
         name: "fake-empty-dir",
@@ -111,7 +111,7 @@ describe("GitFileSystemService", () => {
         size: 0,
         addedTime: fs.statSync(
           `${EFS_VOL_PATH_STAGING}/fake-repo/fake-empty-dir`
-        ).birthtimeMs,
+        ).ctimeMs,
       }
       const expectedAnotherFakeFile: GitDirectoryItem = {
         name: "another-fake-file",
@@ -121,7 +121,7 @@ describe("GitFileSystemService", () => {
         size: "Another fake content".length,
         addedTime: fs.statSync(
           `${EFS_VOL_PATH_STAGING}/fake-repo/another-fake-file`
-        ).birthtimeMs,
+        ).ctimeMs,
       }
 
       const result = await GitFileSystemService.listDirectoryContents(
@@ -162,7 +162,7 @@ describe("GitFileSystemService", () => {
         path: "fake-dir",
         size: 0,
         addedTime: fs.statSync(`${EFS_VOL_PATH_STAGING}/fake-repo/fake-dir`)
-          .birthtimeMs,
+          .ctimeMs,
       }
       const expectedAnotherFakeFile: GitDirectoryItem = {
         name: "another-fake-file",
@@ -172,7 +172,7 @@ describe("GitFileSystemService", () => {
         size: "Another fake content".length,
         addedTime: fs.statSync(
           `${EFS_VOL_PATH_STAGING}/fake-repo/another-fake-file`
-        ).birthtimeMs,
+        ).ctimeMs,
       }
 
       const result = await GitFileSystemService.listDirectoryContents(
@@ -1429,7 +1429,7 @@ describe("GitFileSystemService", () => {
         ).toString("base64")}`,
         mediaPath: "fake-dir/fake-media-file.png",
         type: "file",
-        addedTime: fileStats.birthtimeMs,
+        addedTime: fileStats.ctimeMs,
         size: fileStats.size,
       }