Bound the jitter

open-chat-labs · Dec 18, 2024 · 12cda7c · 12cda7c
1 parent 1b1f848
commit 12cda7c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/backend/canisters/community/impl/src/updates/claim_prize.rs b/backend/canisters/community/impl/src/updates/claim_prize.rs
@@ -102,7 +102,8 @@ fn prepare(args: &Args, state: &mut RuntimeState) -> Result<PrepareResult, Box<R
     };
 
     // Hack to ensure 2 prizes claimed by the same user in the same block don't result in "duplicate transaction" errors.
-    let transaction_time = now_nanos - u32::from(result.message_index) as u64;
+    let jitter = u32::from(result.message_index) as u64 % 1000;
+    let transaction_time = now_nanos - jitter;
 
     let transaction = create_pending_transaction(
         result.token,

diff --git a/backend/canisters/group/impl/src/updates/claim_prize.rs b/backend/canisters/group/impl/src/updates/claim_prize.rs
@@ -87,7 +87,8 @@ fn prepare(args: &Args, state: &mut RuntimeState) -> Result<PrepareResult, Box<R
         };
 
         // Hack to ensure 2 prizes claimed by the same user in the same block don't result in "duplicate transaction" errors.
-        let transaction_time = now_nanos - u32::from(result.message_index) as u64;
+        let jitter = u32::from(result.message_index) as u64 % 1000;
+        let transaction_time = now_nanos - jitter;
 
         let transaction = create_pending_transaction(
             result.token,