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.
This commit is contained in:
parent
069f7716db
commit
047576273c
|
@ -853,7 +853,7 @@ where
|
||||||
if tx_ids.iter().all(|item| item.unmined_tx_id().is_some())
|
if tx_ids.iter().all(|item| item.unmined_tx_id().is_some())
|
||||||
&& !tx_ids.is_empty() =>
|
&& !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"));
|
self.fail_with(PeerError::WrongMessage("inv with mixed item types"));
|
||||||
|
|
Loading…
Reference in New Issue