fix(sanitize): assert number for timestamp (#281)

ar-io · Dec 17, 2024 · 76d7064 · 76d7064
2 parents 84afa76 + 5fe672c
commit 76d7064
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/main.lua b/src/main.lua
@@ -388,7 +388,7 @@ end
 -- Sanitize inputs before every interaction
 local function assertAndSanitizeInputs(msg)
 	assert(
-		msg.Timestamp and msg.Timestamp >= LastKnownMessageTimestamp,
+		msg.Timestamp and tonumber(msg.Timestamp) >= LastKnownMessageTimestamp,
 		"Timestamp must be greater than or equal to the last known message timestamp of "
 			.. LastKnownMessageTimestamp
 			.. " but was "

diff --git a/tests/monitor/monitor.test.mjs b/tests/monitor/monitor.test.mjs
@@ -65,10 +65,13 @@ describe('setup', () => {
       );
       const evalIndex = handlersList.indexOf('_eval');
       const defaultIndex = handlersList.indexOf('_default');
+      const sanitizeIndex = handlersList.indexOf('sanitize');
       const pruneIndex = handlersList.indexOf('prune');
       assert(
-        pruneIndex > evalIndex && pruneIndex === defaultIndex + 1,
-        `Prune index (${pruneIndex}) is not the first handler after _default (${defaultIndex + 1})`,
+        pruneIndex > sanitizeIndex &&
+          sanitizeIndex > evalIndex &&
+          pruneIndex === defaultIndex + 1,
+        `Prune index (${pruneIndex}) and sanitize index (${sanitizeIndex}) are not the first and second handlers after _default (${defaultIndex + 1})`,
       );
     });
   });