diff --git a/CHANGES.md b/CHANGES.md index 763b18ff015..1c30112682c 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -51,6 +51,17 @@ Released on November 13, 2024. [#3974]: https://github.com/planetarium/libplanet/pull/3974 +Version 5.3.2 +------------- + +Released on November 21, 2024 + + - There is no longer a check at block validation time to see + if the actions contained in a block are interpretable. [[#3973]] + +[#3973]: https://github.com/planetarium/libplanet/pull/3973 + + Version 5.3.1 ------------- diff --git a/src/Libplanet/Blockchain/BlockChain.Validate.cs b/src/Libplanet/Blockchain/BlockChain.Validate.cs index e6b1a712580..2100f58eef6 100644 --- a/src/Libplanet/Blockchain/BlockChain.Validate.cs +++ b/src/Libplanet/Blockchain/BlockChain.Validate.cs @@ -4,7 +4,6 @@ using System.Linq; using System.Numerics; using System.Security.Cryptography; -using Bencodex.Types; using Libplanet.Action; using Libplanet.Action.State; using Libplanet.Common; @@ -196,17 +195,6 @@ internal Dictionary
ValidateBlockNonces( return nonceDeltas; } - internal void ValidateBlockLoadActions(Block block) - { - foreach (Transaction tx in block.Transactions) - { - foreach (IValue rawAction in tx.Actions) - { - _ = ActionEvaluator.ActionLoader.LoadAction(block.Index, rawAction); - } - } - } - internal void ValidateBlock(Block block) { if (block.Index <= 0) diff --git a/src/Libplanet/Blockchain/BlockChain.cs b/src/Libplanet/Blockchain/BlockChain.cs index 856403dd71e..fa1fe6ce24f 100644 --- a/src/Libplanet/Blockchain/BlockChain.cs +++ b/src/Libplanet/Blockchain/BlockChain.cs @@ -813,8 +813,6 @@ internal void Append( .ToDictionary(signer => signer, signer => Store.GetTxNonce(Id, signer)), block); - ValidateBlockLoadActions(block); - if (Policy.ValidateNextBlock(this, block) is { } bpve) { throw bpve; diff --git a/test/Libplanet.Tests/Blockchain/BlockChainTest.Append.cs b/test/Libplanet.Tests/Blockchain/BlockChainTest.Append.cs index abe128a4533..77495c89058 100644 --- a/test/Libplanet.Tests/Blockchain/BlockChainTest.Append.cs +++ b/test/Libplanet.Tests/Blockchain/BlockChainTest.Append.cs @@ -651,56 +651,6 @@ void AssertTxIdSetEqual( _blockChain.StagePolicy.Iterate(_blockChain, filtered: false).Select(tx => tx.Id)); } - [SkippableFact] - public void CannotAppendBlockWithInvalidActions() - { - var txSigner = new PrivateKey(); - var unsignedInvalidTx = new UnsignedTx( - new TxInvoice( - _blockChain.Genesis.Hash, - DateTimeOffset.UtcNow, - new TxActionList((IValue)List.Empty.Add(new Text("Foo")))), // Invalid action - new TxSigningMetadata(txSigner.PublicKey, 1)); - var invalidTx = new Transaction( - unsignedInvalidTx, unsignedInvalidTx.CreateSignature(txSigner)); - var txs = new[] - { - Transaction.Create( - nonce: 0, - privateKey: txSigner, - genesisHash: _blockChain.Genesis.Hash, - actions: Array.Empty