feat: add pending connection limits + bump libp2p to `0.54.1`
Linked Issues/PRs
- Closes https://github.com/FuelLabs/fuel-core/issues/1560
- Closes https://github.com/FuelLabs/fuel-core/issues/904
Description
This is apart of the broader initiative outlined in https://github.com/FuelLabs/fuel-core/issues/1968 to improve the DoS resilience
Checklist
- [x] Breaking changes are clearly marked as such in the PR description and changelog
Before requesting review
- [x] I have reviewed the code myself
Looks good to me. Little nit on the CHANGELOG. Can you update it now that you have a new parameter added ?
Good catch @AurelienFT , I've updated it in 1127c7200
We shouldn't merge this PR before https://github.com/FuelLabs/fuel-core/pull/2131
Plus, let's wait for my review as well, since maybe we need to clean up some other logic
How's this coming?
How's this coming?
Thanks for resurfacing this @MitchTurner it got lost in the ether with the mainnet whirlwind of changes.
I've resolved the conflicts so it's ready for review again. I know @xgreenx wanted to look into it again but the changes imo are relatively low surface - I've upgraded our libp2p version + introduced connection limit options.
Hello @maschad,
As you mentionned that @xgreenx wanted to look again we will wait him before merging this but I can already re-approve as a first approver. However before approving the conflicts should be resolved, are you ok to resolve them ? Or we can do it for you, if you prefer :)
Thanks again for this PR.
Thanks @AurelienFT I've gone ahead and resolved the conflicts.
@maschad is it okay if I take over this PR? there seem to be some formatting issues
Thanks for the contribution! Before we can merge this, we need @maschad to sign the Fuel Labs Contributor License Agreement.