v1.2.62

CHANGELOG.md

@@ -1,3 +1,7 @@
+## v1.2.62
+
+- fix throw error on empty files
+
 ## v1.2.61
 
 - fix missing cors origin header when port and crossDomainPort are the same

package.json

@@ -1,6 +1,6 @@
 {
   "name": "rammerhead",
-  "version": "1.2.61",
+  "version": "1.2.62",
   "description": "User friendly web proxy powered by testcafe-hammerhead",
   "main": "src/index.js",
   "scripts": {

src/classes/RammerheadJSFileCache.js

@@ -36,7 +36,7 @@ class RammerheadJSFileCache {
     this.lruMarker = new LRUCache({
       max: maxItems,
       maxSize: jsCacheSize,
-      sizeCalculation: n => n,
+      sizeCalculation: n => n || 1,
       dispose(_, key) {
         fs.unlinkSync(path.join(diskJsCachePath, key));
       }
@@ -55,6 +55,11 @@ class RammerheadJSFileCache {
       initFileList.sort((a, b) => a.size - b.size);
 
       for (const file of initFileList) {
+        if (!file.size) {
+          // writing probably got interrupted. so we delete the corrupted file
+          fs.unlinkSync(path.join(diskJsCachePath, file.key));
+          continue;
+        }
         this.lruMarker.set(file.key, file.size, {
           noDisposeOnSet: true
         });

src/classes/RammerheadJSMemCache.js

@@ -4,7 +4,7 @@ class RammerheadJSMemCache {
   constructor(jsCacheSize) {
     this.lru = new LRUCache({
       maxSize: jsCacheSize,
-      sizeCalculation: n => n.length
+      sizeCalculation: n => n.length || 1
     });
   }
   get(key) {

src/util/addJSDiskCache.js

@@ -15,7 +15,10 @@ module.exports = async function (jsCache) {
     const md5 = (data) => crypto.createHash('md5').update(data).digest('hex');
 
     cacheGet = async (key) => await jsCache.get(md5(key));
-    cacheSet = async (key, value) => await jsCache.set(md5(key), value);
+    cacheSet = async (key, value) => {
+        if (!value) return;
+        await jsCache.set(md5(key), value);
+    }
 };
 
 // patch ScriptResourceProcessor