fixes EIP2718/EIP2930 misinterpretation

the new Tx format (txType || txPayload) needs envelope
encoding at EthBlock level, not at the Tx level itself.

eth/common/eth_types.nim

@@ -154,7 +154,7 @@ type
 
   EthBlock* = object
     header*: BlockHeader
-    txs*   : seq[Transaction]
+    txs* {.rlpCustomSerialization.}: seq[Transaction]
     uncles*: seq[BlockHeader]
 
   CollationHeader* = object
@@ -355,29 +355,48 @@ proc append*(rlpWriter: var RlpWriter, tx: Transaction) =
   if tx.txType == LegacyTxType:
     rlpWriter.append(tx.legacyTx)
   else:
-    var rw = initRlpWriter()
+    # EIP 2718/2930
-    rw.append(1)
+    rlpWriter.append(1)
-    rw.append(tx.accessListTx)
+    rlpWriter.append(tx.accessListTx)
-    rlpWriter.append(rw.finish())
 
 proc read*(rlp: var Rlp, T: type Transaction): T =
   if rlp.isList:
-    result = Transaction(
+    return Transaction(
       txType: LegacyTxType,
       legacyTx: rlp.read(LegacyTx)
     )
-  else:
+
-    let bytes = rlp.read(Blob)
+  # EIP 2718/2930
-    var rr = rlpFromBytes(bytes)
+  let txType = rlp.read(int)
-    let txType = rr.read(int)
   if txType != 1:
     raise newException(UnsupportedRlpError,
       "TxType expect 1 got " & $txType)
-    result = Transaction(
+  return Transaction(
     txType: AccessListTxType,
-      accessListTx: rr.read(AccessListTx)
+    accessListTx: rlp.read(AccessListTx)
   )
 
+proc read*(rlp: var Rlp, t: var EthBlock, _: type seq[Transaction]): seq[Transaction] {.inline.} =
+  # EIP 2718/2930: we have to override this field
+  # for reasons described below in `append` proc
+  for tx in rlp:
+    if tx.isList:
+      result.add tx.read(Transaction)
+    else:
+      let bytes = rlp.read(Blob)
+      var rr = rlpFromBytes(bytes)
+      result.add rr.read(Transaction)
+
+proc append*(rlpWriter: var RlpWriter, blk: EthBlock, txs: seq[Transaction]) {.inline.} =
+  # EIP 2718/2930: the new Tx is rlp(txType || txPlayload) -> one blob/one list elem
+  # not rlp(txType, txPayload) -> two list elem, wrong!
+  rlpWriter.startList(txs.len)
+  for tx in txs:
+    if tx.txType == LegacyTxType:
+      rlpWriter.append(tx)
+    else:
+      rlpWriter.append(rlp.encode(tx))
+
 proc read*(rlp: var Rlp, T: type HashOrStatus): T {.inline.} =
   if rlp.isBlob() and (rlp.blobLen() == 32 or rlp.blobLen() == 1):
     if rlp.blobLen == 1:
@@ -416,27 +435,25 @@ proc append*(rlpWriter: var RlpWriter, rec: Receipt) =
   if rec.receiptType == LegacyReceiptType:
     rlpWriter.append(rec.legacyReceipt)
   else:
-    var rw = initRlpWriter()
+    # EIP 2718/2930
-    rw.append(1)
+    rlpWriter.append(1)
-    rw.append(rec.accessListReceipt)
+    rlpWriter.append(rec.accessListReceipt)
-    rlpWriter.append(rw.finish())
 
 proc read*(rlp: var Rlp, T: type Receipt): T =
   if rlp.isList:
-    result = Receipt(
+    return Receipt(
       receiptType: LegacyReceiptType,
       legacyReceipt: rlp.read(LegacyReceipt)
     )
-  else:
+
-    let bytes = rlp.read(Blob)
+  # EIP 2718/2930
-    var rr = rlpFromBytes(bytes)
+  let recType = rlp.read(int)
-    let recType = rr.read(int)
   if recType != 1:
     raise newException(UnsupportedRlpError,
       "TxType expect 1 got " & $recType)
-    result = Receipt(
+  return Receipt(
     receiptType: AccessListReceiptType,
-      accessListReceipt: rr.read(AccessListReceipt)
+    accessListReceipt: rlp.read(AccessListReceipt)
   )
 
 proc read*(rlp: var Rlp, T: type Time): T {.inline.} =

eth/common/transaction.nim

@@ -88,6 +88,7 @@ func rlpEncodeEIP155*(tx: LegacyTx): auto =
     ))
 
 func rlpEncode*(tx: AccessListTx): auto =
+  # EIP 2718/2930
   let unsignedTx = AccessListUnsignedTx(
     chainId: tx.chainId,
     nonce: tx.nonce,
@@ -102,7 +103,7 @@ func rlpEncode*(tx: AccessListTx): auto =
   var rw = initRlpWriter()
   rw.append(1)
   rw.append(unsignedTx)
-  rlp.encode(rw.finish())
+  rw.finish()
 
 func txHashNoSignature*(tx: LegacyTx): Hash256 =
   # Hash transaction without signature