Finalize domain block only when the new finalized block is higher tha…

…n last finalized Ref paritytech/substrate#14347
autonomys · Jun 13, 2023 · 12c7062 · 12c7062
1 parent 38decf2
commit 12c7062
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/domains/client/domain-executor/src/domain_block_processor.rs b/domains/client/domain-executor/src/domain_block_processor.rs
@@ -236,17 +236,17 @@ where
         if let Some(to_finalize_block_number) =
             header_number.checked_sub(&self.domain_confirmation_depth)
         {
-            let to_finalize_block_hash =
-                self.client.hash(to_finalize_block_number)?.ok_or_else(|| {
-                    sp_blockchain::Error::Backend(format!(
-                        "Header for #{to_finalize_block_number} not found"
-                    ))
-                })?;
-            self.client
-                .finalize_block(to_finalize_block_hash, None, true)?;
-            tracing::debug!(
-                "Successfully finalized block: #{to_finalize_block_number},{to_finalize_block_hash}"
-            );
+            if to_finalize_block_number > self.client.info().finalized_number {
+                let to_finalize_block_hash =
+                    self.client.hash(to_finalize_block_number)?.ok_or_else(|| {
+                        sp_blockchain::Error::Backend(format!(
+                            "Header for #{to_finalize_block_number} not found"
+                        ))
+                    })?;
+                self.client
+                    .finalize_block(to_finalize_block_hash, None, true)?;
+                tracing::debug!("Successfully finalized block: #{to_finalize_block_number},{to_finalize_block_hash}");
+            }
         }
 
         let mut roots = self.client.runtime_api().intermediate_roots(header_hash)?;