Add new ValidateRejectCodes that are more descriptive and update failure to find a parent block id as an UnknownParent error code

Signed-off-by: Jacinta Ferrant <jacinta.ferrant@gmail.com>

Author: jferrant

stacks-signer/src/v0/signer.rs

@@ -1541,8 +1541,7 @@ fn should_reevaluate_block(block_info: &BlockInfo) -> bool {
             | RejectReason::TestingDirective
             | RejectReason::InvalidTenureExtend
             | RejectReason::NotRejected
-            | RejectReason::Unknown(_)
-            | RejectReason::InvalidParentBlock => true,
+            | RejectReason::Unknown(_) => true,
             RejectReason::ValidationFailed(_)
             | RejectReason::RejectedInPriorRound
             | RejectReason::SortitionViewMismatch
@@ -1551,6 +1550,7 @@ fn should_reevaluate_block(block_info: &BlockInfo) -> bool {
             | RejectReason::PubkeyHashMismatch
             | RejectReason::InvalidMiner
             | RejectReason::NotLatestSortitionWinner
+            | RejectReason::InvalidParentBlock
             | RejectReason::DuplicateBlockFound => {
                 // No need to re-validate these types of rejections.
                 false

stackslib/src/net/api/postblock_proposal.rs

@@ -87,7 +87,10 @@ define_u8_enum![ValidateRejectCode {
     ChainstateError = 3,
     UnknownParent = 4,
     NonCanonicalTenure = 5,
-    NoSuchTenure = 6
+    NoSuchTenure = 6,
+    InvalidParentBlock = 7,
+    InvalidTimestamp = 8,
+    NetworkChainMismatch = 9
 }];
 
 pub static TOO_MANY_REQUESTS_STATUS: u16 = 429;
@@ -287,7 +290,7 @@ impl NakamotoBlockProposal {
                 "highest_header.height" => highest_header.anchored_header.height(),
             );
             return Err(BlockValidateRejectReason {
-                reason_code: ValidateRejectCode::InvalidBlock,
+                reason_code: ValidateRejectCode::InvalidParentBlock,
                 reason: "Block is not higher than the highest block in its tenure".into(),
             });
         }
@@ -405,7 +408,7 @@ impl NakamotoBlockProposal {
                 "received_mainnet" => mainnet,
             );
             return Err(BlockValidateRejectReason {
-                reason_code: ValidateRejectCode::InvalidBlock,
+                reason_code: ValidateRejectCode::NetworkChainMismatch,
                 reason: "Wrong network/chain_id".into(),
             });
         }
@@ -428,8 +431,8 @@ impl NakamotoBlockProposal {
             &self.block.header.parent_block_id,
         )?
         .ok_or_else(|| BlockValidateRejectReason {
-            reason_code: ValidateRejectCode::InvalidBlock,
-            reason: "Invalid parent block".into(),
+            reason_code: ValidateRejectCode::UnknownParent,
+            reason: "Unknown parent block".into(),
         })?;
 
         let burn_view_consensus_hash =
@@ -494,7 +497,7 @@ impl NakamotoBlockProposal {
                     "parent_block_timestamp" => parent_nakamoto_header.timestamp,
                 );
                 return Err(BlockValidateRejectReason {
-                    reason_code: ValidateRejectCode::InvalidBlock,
+                    reason_code: ValidateRejectCode::InvalidTimestamp,
                     reason: "Block timestamp is not greater than parent block".into(),
                 });
             }
@@ -507,7 +510,7 @@ impl NakamotoBlockProposal {
                 "current_time" => get_epoch_time_secs(),
             );
             return Err(BlockValidateRejectReason {
-                reason_code: ValidateRejectCode::InvalidBlock,
+                reason_code: ValidateRejectCode::InvalidTimestamp,
                 reason: "Block timestamp is too far into the future".into(),
             });
         }

stackslib/src/net/api/tests/postblock_proposal.rs

@@ -473,7 +473,7 @@ fn test_try_make_response() {
             reason,
             ..
         }) => {
-            assert_eq!(reason_code, ValidateRejectCode::InvalidBlock);
+            assert_eq!(reason_code, ValidateRejectCode::InvalidTimestamp);
             assert_eq!(reason, "Block timestamp is not greater than parent block");
         }
     }
@@ -486,7 +486,7 @@ fn test_try_make_response() {
             reason,
             ..
         }) => {
-            assert_eq!(reason_code, ValidateRejectCode::InvalidBlock);
+            assert_eq!(reason_code, ValidateRejectCode::InvalidTimestamp);
             assert_eq!(reason, "Block timestamp is too far into the future");
         }
     }

testnet/stacks-node/src/tests/nakamoto_integrations.rs

@@ -3133,7 +3133,7 @@ fn block_proposal_api_endpoint() {
                 sign(&p)
             },
             HTTP_ACCEPTED,
-            Some(Err(ValidateRejectCode::InvalidBlock)),
+            Some(Err(ValidateRejectCode::NetworkChainMismatch)),
         ),
         (
             "Invalid `miner_signature`",

testnet/stacks-node/src/tests/signer/v0.rs

@@ -1511,7 +1511,7 @@ fn block_proposal_rejection() {
         signer_test.wait_for_validate_reject_response(short_timeout, block_signer_signature_hash_2);
     assert!(matches!(
         reject.reason_code,
-        ValidateRejectCode::InvalidBlock
+        ValidateRejectCode::UnknownParent
     ));
 
     let start_polling = Instant::now();
@@ -1541,7 +1541,7 @@ fn block_proposal_rejection() {
                     found_signer_signature_hash_2 = true;
                     assert!(matches!(
                         reason_code,
-                        RejectCode::ValidationFailed(ValidateRejectCode::InvalidBlock)
+                        RejectCode::ValidationFailed(ValidateRejectCode::UnknownParent)
                     ));
                 } else {
                     continue;