nim-eth/eth/p2p/rlpx_protocols
Jamie Lokier 5234e30f8b Remove `{.rlpInline.}` which was never really implemented
For a long time this caused invalid RLP parsing of `NewBlock` messages in the
`eth` protocol.

The `rlpInline` pragma was accepted but had no effect.  We could implemented
it, but it doesn't seem worth doing, with tests etc, as there's only one user
which has been fixed another way.

With `NewBlock`, whenever a peer sent us `NewBlock`, we'd get an RLP decoding
error, and disconnected the peer thinking it was the peer's error.

These messages are sent often by good peers, so whenever we connected to a
really good peer, we'd end up disconnecting within a minute due to this.  This
went unnoticed for years, as we stayed connected to old peers which have no new
blocks, and we weren't looking at peer quality, disconnect reasons or real-time
blockchain updates anyway.

Signed-off-by: Jamie Lokier <jamie@shareable.org>
2021-08-10 14:08:35 +03:00
..
les Fix raw Exceptions in hexary caused by forward declarations (#349) 2021-05-11 17:32:47 +02:00
whisper Add top level push raises Defect to p2p code (#374) 2021-07-16 21:44:30 +02:00
bzz_protocol.nim Add build_dcli target and add it to CI (#344) 2021-04-06 13:33:24 +02:00
eth_protocol.nim Remove `{.rlpInline.}` which was never really implemented 2021-08-10 14:08:35 +03:00
les_protocol.nim Fix import syntax error in `les_protocol.nim` added by PR #344 2021-04-07 18:07:00 +01:00
whisper_protocol.nim Add top level push raises Defect to p2p code (#374) 2021-07-16 21:44:30 +02:00