Version 0.8.6

Fixed issue with storage migration script, where `--dryrun` mode was causing streams to stack up and eventually crash with EMFILE: too many open files.
jhuckaby · May 30, 2018 · 06a6e6c · 06a6e6c
1 parent e13b8fc
commit 06a6e6c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/bin/storage-migrate.js b/bin/storage-migrate.js
@@ -346,9 +346,13 @@ var StorageMigrator = {
 						}
 
 						if (args.dryrun) {
-							verbose("DRY RUN: Copied binary record: " + key + "\n");
-							self.numRecords++;
-							return callback();
+							stream.on('end', function() {
+								verbose("DRY RUN: Copied binary record: " + key + "\n");
+								self.numRecords++;
+								callback();
+							});
+							stream.resume();
+							return;
 						}
 
 						self.newStorage.putStream(key, stream, function(err) {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
 	"name": "Cronicle",
-	"version": "0.8.5",
+	"version": "0.8.6",
 	"description": "A simple, distributed task scheduler and runner with a web based UI.",
 	"author": "Joseph Huckaby <[email protected]>",
 	"homepage": "https://github.com/jhuckaby/Cronicle",