Fixed possible race condition in fileHasChanged function.

eklingen · Dec 12, 2024 · 45a6781 · 45a6781
1 parent 6afab1b
commit 45a6781
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,7 @@
 
 # Changelog
 
+v1.1.3 - Fixed possible race condition in `fileHasChanged` function.
 v1.1.2 - Some small changes for Windows compatibility
 v1.1.1 - Added `enableBypass` option to the constructor, to enable quick disabling of the whole caching mechanism.
 v1.1.0 - Addd `removeEntriesByKeys(...key)` function.

diff --git a/index.js b/index.js
@@ -107,12 +107,13 @@ export default class FileHashCache {
       return true
     }
 
+    const fileKey = relativePath(this.projectRoot, filepath)
+    const fileHash = await this.#getFileHash(filepath, encoding)
+
     if (!this.hashCache[key]) {
       await this.load(key)
     }
 
-    const fileKey = relativePath(this.projectRoot, filepath)
-    const fileHash = await this.#getFileHash(filepath, encoding)
     const cachedHash = this.hashCache[key][fileKey] || ''
     const fileHasChanged = (fileHash !== cachedHash)