Add safegoard for out of control loop on internal error.

Hexagon · Jan 6, 2024 · c649a69 · c649a69
1 parent e723717
commit c649a69
Show file tree

Hide file tree

Showing 14 changed files with 98 additions and 85 deletions.
diff --git a/README.md b/README.md
@@ -68,13 +68,13 @@ const { Cron } = require("croner");
 Using Deno
 
 ```typescript
-import { Cron } from "https://deno.land/x/croner@8.0.0/dist/croner.js";
+import { Cron } from "https://deno.land/x/croner@8.0.1/dist/croner.js";
 ```
 
 In a webpage using the UMD-module
 
 ```html
-<script src="https://cdn.jsdelivr.net/npm/croner@7/dist/croner.umd.min.js"></script>
+<script src="https://cdn.jsdelivr.net/npm/croner@8/dist/croner.umd.min.js"></script>
 ```
 
 ## Documentation

diff --git a/dist/croner.cjs b/dist/croner.cjs
diff --git a/dist/croner.js b/dist/croner.js
diff --git a/dist/croner.min.cjs b/dist/croner.min.cjs
diff --git a/dist/croner.min.cjs.map b/dist/croner.min.cjs.map
diff --git a/dist/croner.min.js b/dist/croner.min.js
diff --git a/dist/croner.min.js.map b/dist/croner.min.js.map
diff --git a/dist/croner.umd.js b/dist/croner.umd.js
diff --git a/dist/croner.umd.min.js b/dist/croner.umd.min.js
diff --git a/dist/croner.umd.min.js.map b/dist/croner.umd.min.js.map
diff --git a/docs/src/_data.json b/docs/src/_data.json
@@ -6,7 +6,7 @@
       "description": "Cron for JavaScript and TypeScript"
     },
     "substitute": {
-      "$CRONER_VERSION": "8.0.0"
+      "$CRONER_VERSION": "8.0.1"
     },
     "top_links": [
       {

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "croner",
-  "version": "8.0.0",
+  "version": "8.0.1-dev.0",
   "description": "Trigger functions and/or evaluate cron expressions in JavaScript. No dependencies. Most features. All environments.",
   "author": "Hexagon <github.com/hexagon>",
   "homepage": "https://hexagon.github.io/croner",

diff --git a/src/croner.js b/src/croner.js
@@ -391,7 +391,8 @@ Cron.prototype.schedule = function (func) {
 	// Get the target date based on previous run
 	const target = this.nextRun(this._states.currentRun);
 
-	if (waitMs === null || target === null) return this;
+	// isNaN added to prevent infinite loop
+	if (waitMs === null || waitMs === undefined || isNaN(waitMs) || target === null) return this;
 
 	// setTimeout cant handle more than Math.pow(2, 32 - 1) - 1 ms
 	if (waitMs > maxDelay) {