net: Avoid logging AlreadyHaveTx when disconnecting misbehaving peer #23042

1. 
   MarcoFalke commented at 7:24 am on September 20, 2021: member

   There is no need to log AlreadyHaveTx for an inv when a peer is marked for disconnection due to sending that inv. In fact, I find it confusing that a block-relay-only connection calls AlreadyHaveTx at all. Also there is no need to call AddKnownTx when the peer is marked for disconnection.
2. net: Avoid logging AlreadyHaveTx when disconnecting misbehaving peer

   Can be reviewed with --color-moved=dimmed-zebra

   fa2662c293
3. MarcoFalke renamed this:
   net: Avoid logging AlreadyHaveTx when disconneting misbehaving peer
   net: Avoid logging AlreadyHaveTx when disconnecting misbehaving peer
   on Sep 20, 2021
4. MarcoFalke force-pushed on Sep 20, 2021
5. fanquake added the label P2P on Sep 20, 2021
6. 
   naumenkogs commented at 10:28 am on September 21, 2021: member

   ACK fa2662c293ec0aaa93092b59b6632f74729c4283
7. 
   jnewbery commented at 2:45 pm on September 21, 2021: member

   Code review ACK fa2662c293ec0aaa93092b59b6632f74729c4283
8. fanquake requested review from amitiuttarwar on Sep 23, 2021
9. 
   DrahtBot commented at 1:30 pm on September 24, 2021: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #22778 (net processing: Reduce resource usage for inbound block-relay-only connections by jnewbery)
   • #21160 (net/net processing: Move tx inventory into net_processing by jnewbery)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

10. 
    dunxen commented at 6:45 pm on October 5, 2021: contributor

    Concept and code review ACK fa2662c293ec0aaa93092b59b6632f74729c4283

    Makes sense to return earlier in this scenario. This took me on a nice little journey in P2P things getting some better context.

11. fanquake merged this on Oct 22, 2021
12. fanquake closed this on Oct 22, 2021

    ---

13. MarcoFalke deleted the branch on Oct 22, 2021
14. sidhujag referenced this in commit f61e177b1d on Oct 22, 2021
15. DrahtBot locked this on Oct 30, 2022

Contributors
MarcoFalke naumenkogs jnewbery DrahtBot dunxen

Review Requested
amitiuttarwar

Labels
P2P