fix mtp check?

keep-starknet-strange · Oct 21, 2024 · d87d425 · d87d425
1 parent 11557e9
commit d87d425
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/packages/consensus/src/types/chain_state.cairo b/packages/consensus/src/types/chain_state.cairo
@@ -64,8 +64,9 @@ pub impl BlockValidatorImpl of BlockValidator {
         let block_height = self.block_height + 1;
 
         let prev_block_time = *self.prev_timestamps[self.prev_timestamps.len() - 1];
+
+        let median_time_past = compute_median_time_past(self.prev_timestamps);
         let prev_timestamps = next_prev_timestamps(self.prev_timestamps, block.header.time);
-        let median_time_past = compute_median_time_past(prev_timestamps);
 
         validate_timestamp(median_time_past, block.header.time)?;
 

diff --git a/packages/consensus/src/validation/timestamp.cairo b/packages/consensus/src/validation/timestamp.cairo
@@ -45,7 +45,13 @@ pub fn validate_timestamp(median_time_past: u32, block_time: u32) -> Result<(),
     if block_time > median_time_past {
         Result::Ok(())
     } else {
-        Result::Err("Median time past is greater than or equal to block's timestamp")
+        Result::Err(
+            format!(
+                "Median time past: {} >= block's timestamp: {}.",
+                median_time_past,
+                block_time
+            )
+        )
     }
 }