Merge pull request #2079 from mintlayer/consensus_tightening

Consensus tightening

Author: ImplOfAnImpl

CHANGELOG.md

@@ -45,6 +45,14 @@ The format is loosely based on [Keep a Changelog](https://keepachangelog.com/en/
       of a cluster that exceeds the limits. The default cluster limits are: max 64 transactions, max 100'000 bytes
       in total.
 
+  - Consensus:
+    - Tightening of certain consensus rules have been implemented, in particular:
+      - When changing token metadata URI, the URI can no longer contain arbitrary characters (the restrictions are
+        the same as when a token is created).
+      - Transferring or burning zero amount of a token is no longer allowed.
+
+      (The actual consensus tightening will happen after a fork, the height is yet to be decided.)
+
 ## [1.3.1] - 2026-06-03
 
 ### Fixed

chainstate/src/detail/ban_score.rs

@@ -144,6 +144,7 @@ impl BanScore for ConnectTransactionError {
             ConnectTransactionError::ConcludeInputAmountsDontMatch(_, _) => 100,
             ConnectTransactionError::ProduceBlockFromStakeChangesStakerDestination(_, _) => 100,
             ConnectTransactionError::IdCreationError(err) => err.ban_score(),
+            ConnectTransactionError::ZeroTokenTransfer(_) => 100,
         }
     }
 }
@@ -358,6 +359,7 @@ impl BanScore for TokensError {
             TokensError::InvariantBrokenUndoIssuanceOnNonexistentToken(_) => 100,
             TokensError::InvariantBrokenRegisterIssuanceWithDuplicateId(_) => 100,
             TokensError::TokenMetadataUriTooLarge(_) => 100,
+            TokensError::IncorrectMetadataUri(_) => 100,
         }
     }
 }
@@ -476,6 +478,7 @@ impl BanScore for ConsensusPoSError {
             ConsensusPoSError::EmptyTimespan => 100,
             ConsensusPoSError::FailedToCalculateCappedBalance => 100,
             ConsensusPoSError::InvalidOutputTypeInStakeKernel(_) => 100,
+            ConsensusPoSError::PoolIdsInKernelUtxoAndPoSDataMismatch { .. } => 100,
         }
     }
 }

chainstate/src/detail/error_classification.rs

@@ -304,9 +304,8 @@ impl BlockProcessingErrorClassification for ConnectTransactionError {
             | ConnectTransactionError::InsufficientCoinsFee(_, _)
             | ConnectTransactionError::AttemptToSpendFrozenToken(_)
             | ConnectTransactionError::ConcludeInputAmountsDontMatch(_, _)
-            | ConnectTransactionError::ProduceBlockFromStakeChangesStakerDestination(_, _) => {
-                BlockProcessingErrorClass::BadBlock
-            }
+            | ConnectTransactionError::ProduceBlockFromStakeChangesStakerDestination(_, _)
+            | ConnectTransactionError::ZeroTokenTransfer(_) => BlockProcessingErrorClass::BadBlock,
 
             ConnectTransactionError::StorageError(err) => err.classify(),
             ConnectTransactionError::UtxoError(err) => err.classify(),
@@ -494,6 +493,7 @@ impl BlockProcessingErrorClassification for TokensError {
             | TokensError::CoinOrTokenOverflow(_)
             | TokensError::InsufficientTokenFees(_)
             | TokensError::TokenMetadataUriTooLarge(_)
+            | TokensError::IncorrectMetadataUri(_)
             | TokensError::InvariantBrokenUndoIssuanceOnNonexistentToken(_)
             | TokensError::InvariantBrokenRegisterIssuanceWithDuplicateId(_) => {
                 BlockProcessingErrorClass::BadBlock
@@ -663,7 +663,8 @@ impl BlockProcessingErrorClassification for ConsensusPoSError {
             | ConsensusPoSError::FiniteTotalSupplyIsRequired
             | ConsensusPoSError::UnsupportedConsensusVersion
             | ConsensusPoSError::FailedToCalculateCappedBalance
-            | ConsensusPoSError::InvalidOutputTypeInStakeKernel(_) => {
+            | ConsensusPoSError::InvalidOutputTypeInStakeKernel(_)
+            | ConsensusPoSError::PoolIdsInKernelUtxoAndPoSDataMismatch { .. } => {
                 BlockProcessingErrorClass::BadBlock
             }
 

chainstate/src/detail/mod.rs

@@ -86,7 +86,7 @@ pub use self::{
     median_time::calculate_median_time_past_from_blocktimestamps,
 };
 pub use chainstate_types::Locator;
-pub use chainstateref::NonZeroPoolBalances;
+pub use chainstateref::{EpochSealError, NonZeroPoolBalances};
 pub use error::{
     BlockError, CheckBlockError, CheckBlockTransactionsError, DbCommittingContext,
     InitializationError, OrphanCheckError, StorageCompatibilityCheckError,

chainstate/src/lib.rs

@@ -39,9 +39,9 @@ pub use crate::{
     detail::{
         BlockError, BlockProcessingErrorClass, BlockProcessingErrorClassification, BlockSource,
         ChainInfo, CheckBlockError, CheckBlockTransactionsError, ConnectTransactionError,
-        IOPolicyError, InitializationError, Locator, MEDIAN_TIME_SPAN, NonZeroPoolBalances,
-        OrphanCheckError, SpendStakeError, StorageCompatibilityCheckError, TokenIssuanceError,
-        TokensError, TransactionVerifierStorageError, ban_score,
+        EpochSealError, IOPolicyError, InitializationError, Locator, MEDIAN_TIME_SPAN,
+        NonZeroPoolBalances, OrphanCheckError, SpendStakeError, StorageCompatibilityCheckError,
+        TokenIssuanceError, TokensError, TransactionVerifierStorageError, ban_score,
         block_invalidation::BlockInvalidatorError, bootstrap::BootstrapError,
         calculate_median_time_past, calculate_median_time_past_from_blocktimestamps,
     },