Decrease peer reputation on request timeouts and decode errors (FuelL…

…abs#1574)

Closes FuelLabs#1345. Closes FuelLabs#1346 Closes FuelLabs#1350.

This PR stops discarding request errors from libp2p, and instead returns
them to the sender of the request. Also penalizes peers for sending
invalid responses or for not replying at all.

Making penalty configurable should be a follow-up PR, as there are other
penalties that should be configurable as well

TODO:
- [x] Make timeout configutable: Already seems to be case on master
branch
- [x] Add tests
- [x] Fix current tests that for some reason don't terminate

---------

Co-authored-by: xgreenx <[email protected]>

CHANGELOG.md

@@ -22,6 +22,7 @@ Description of the upcoming release here.
 - [#1585](https://github.com/FuelLabs/fuel-core/pull/1585): Let `NetworkBehaviour` macro generate `FuelBehaviorEvent` in p2p
 - [#1579](https://github.com/FuelLabs/fuel-core/pull/1579): The change extracts the off-chain-related logic from the executor and moves it to the GraphQL off-chain worker. It creates two new concepts - Off-chain and On-chain databases where the GraphQL worker has exclusive ownership of the database and may modify it without intersecting with the On-chain database.
 - [#1577](https://github.com/FuelLabs/fuel-core/pull/1577): Moved insertion of sealed blocks into the `BlockImporter` instead of the executor.
+- [#1574](https://github.com/FuelLabs/fuel-core/pull/1574): Penalizes peers for sending invalid responses or for not replying at all.
 - [#1601](https://github.com/FuelLabs/fuel-core/pull/1601): Fix formatting in docs and check that `cargo doc` passes in the CI.
 - [#1636](https://github.com/FuelLabs/fuel-core/pull/1636): Add more docs to GraphQL DAP API.
 

crates/services/p2p/src/behavior.rs

@@ -187,7 +187,7 @@ impl FuelBehaviour {
             Ok(true) => {
                 tracing::debug!(target: "fuel-p2p", "Sent a report for MessageId: {} from PeerId: {}", msg_id, propagation_source);
                 if should_check_score {
-                    return self.gossipsub.peer_score(propagation_source)
+                    return self.gossipsub.peer_score(propagation_source);
                 }
             }
             Ok(false) => {