From 047576273c26b3942bb8e783f037033f20f6a0b9 Mon Sep 17 00:00:00 2001 From: teor Date: Wed, 25 Aug 2021 07:40:21 +1000 Subject: [PATCH] Stop converting `Message::Inv(TxId+)` into `Request::TransactionsById` (#2660) `Message::Inv(TxId+)` is a transaction advertisement, so it should be converted into `Request::AdvertiseTransactionIds`. This is a copy-paste mistake from the original zebra-network implementation. --- zebra-network/src/peer/connection.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/zebra-network/src/peer/connection.rs b/zebra-network/src/peer/connection.rs index 6635f49ba26..6010243c050 100644 --- a/zebra-network/src/peer/connection.rs +++ b/zebra-network/src/peer/connection.rs @@ -853,7 +853,7 @@ where if tx_ids.iter().all(|item| item.unmined_tx_id().is_some()) && !tx_ids.is_empty() => { - Request::TransactionsById(transaction_ids(&items).collect()) + Request::AdvertiseTransactionIds(transaction_ids(&items).collect()) } _ => { self.fail_with(PeerError::WrongMessage("inv with mixed item types"));