consensus: fix a problem with script sizes (#396)

A output's script may be bigger than 10k bytes, it just can't be spent.
We incorrectly check the script pubkey size when it gets created,
causing floresta to break out of consensus if such an output is mined.

Fixes #393

crates/floresta-chain/src/pruned_utreexo/consensus.rs

@@ -105,13 +105,12 @@ impl Consensus {
                 continue;
             }
 
-            // Sum tx output amounts, check their locking script sizes (scriptpubkey)
-            let mut out_value = 0;
-            for output in transaction.output.iter() {
-                out_value += output.value.to_sat();
-
-                Self::validate_script_size(&output.script_pubkey, txid)?;
-            }
+            // Sum tx output amounts. This will be used for the fee calculation
+            let out_value: u64 = transaction
+                .output
+                .iter()
+                .map(|out| out.value.to_sat())
+                .sum();
 
             // Sum tx input amounts, check their unlocking script sizes (scriptsig and TODO witness)
             let mut in_value = 0;