Merge pull request #1621 from ebkr/more-file-locks

Lock both source and destination files when copying or renaming files
ebkr · Feb 17, 2025 · eb4dd72 · eb4dd72
2 parents 83f49ca + 1c6db46
commit eb4dd72
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/providers/generic/file/NodeFs.ts b/src/providers/generic/file/NodeFs.ts
@@ -93,7 +93,7 @@ export default class NodeFs extends FsProvider {
 
     async rename(path: string, newPath: string): Promise<void> {
         return new Promise((resolve, reject) => {
-            NodeFs.lock.acquire(path, async () => {
+            NodeFs.lock.acquire([path, newPath], async () => {
                 try {
                     await fs.promises.rename(path, newPath);
                     resolve();
@@ -110,7 +110,7 @@ export default class NodeFs extends FsProvider {
 
     async copyFile(from: string, to: string): Promise<void> {
         return new Promise((resolve, reject) => {
-            NodeFs.lock.acquire(from, async () => {
+            NodeFs.lock.acquire([from, to], async () => {
                 try {
                     await fs.promises.copyFile(from, to);
                     resolve();