Skip to content

Commit

Permalink
chore(validation): LastKnownMessageTimestamp is giving issues for e… (
Browse files Browse the repository at this point in the history
#298)

…xisting observers. For now we will allow anything greater than 0 until
release 23, where they will use the latest SDK.
  • Loading branch information
dtfiedler authored Dec 29, 2024
2 parents 029a449 + b67fe2e commit 8c9dc20
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/main.lua
Original file line number Diff line number Diff line change
Expand Up @@ -387,7 +387,8 @@ end
-- Sanitize inputs before every interaction
local function assertAndSanitizeInputs(msg)
assert(
msg.Timestamp and tonumber(msg.Timestamp) >= LastKnownMessageTimestamp,
-- TODO: replace this with LastKnownMessageTimestamp after node release 23.0.0
msg.Timestamp and tonumber(msg.Timestamp) >= 0,
"Timestamp must be greater than or equal to the last known message timestamp of "
.. LastKnownMessageTimestamp
.. " but was "
Expand Down

0 comments on commit 8c9dc20

Please sign in to comment.