Since the Seoul protocol, reveal operations require the presence of a boolean for whether or not the proof field is present. This is required when manually forging bytes. Added.

Author: csoreff

src/Tezos/Forging.cpp

@@ -179,6 +179,7 @@ Data forgeOperation(const Proto::Operation& operation) {
         append(forged, forgedGasLimit);
         append(forged, forgedStorageLimit);
         append(forged, forgedPublicKey);
+        forged.push_back(0x00); // presence_of_proof = false (no proof for non-tz4)
         return forged;
     }
 

tests/chains/Tezos/ForgingTests.cpp

@@ -226,7 +226,7 @@ TEST(TezosTransaction, forgeReveal) {
     revealOperation.set_kind(TW::Tezos::Proto::Operation::REVEAL);
     revealOperation.set_allocated_reveal_operation_data(revealOperationData);
 
-    auto expected = "6b0081faa75f741ef614b0e35fcc8c90dfa3b0b95721f80992f001f44e810200429a986c8072a40a1f3a3e2ab5a5819bb1b2fb69993c5004837815b9dc55923e";
+    auto expected = "6b0081faa75f741ef614b0e35fcc8c90dfa3b0b95721f80992f001f44e810200429a986c8072a40a1f3a3e2ab5a5819bb1b2fb69993c5004837815b9dc55923e00";
     auto serialized = forgeOperation(revealOperation);
 
     ASSERT_EQ(hex(serialized), expected);