Skip to content

Commit

Permalink
Stop converting Message::Inv(TxId+) into Request::TransactionsById (
Browse files Browse the repository at this point in the history
#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.
  • Loading branch information
teor2345 authored Aug 24, 2021
1 parent 069f771 commit 0475762
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion zebra-network/src/peer/connection.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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"));
Expand Down

0 comments on commit 0475762

Please sign in to comment.