Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add config options for max pending connections, for both outgoing and incoming #904

Open
Voxelot opened this issue Jan 13, 2023 · 0 comments · May be fixed by #2150
Open

Add config options for max pending connections, for both outgoing and incoming #904

Voxelot opened this issue Jan 13, 2023 · 0 comments · May be fixed by #2150
Labels
fuel-p2p tech-debt The issue is to improve the current code and make it more clear/generic/reusable/pretty/avoidable.

Comments

@Voxelot
Copy link
Member

Voxelot commented Jan 13, 2023

          ... this defines the number of max established "incoming" connections, I arbitrarily set it to be 50% of total connections allowed, so 50:50 between incoming and outgoing connections. But we can have this configurable in p2p config if we want, plus maybe we should add config options for max pending connections, for both outgoing and incoming.

Originally posted by @leviathanbeak in #876 (comment)

@xgreenx xgreenx added tech-debt The issue is to improve the current code and make it more clear/generic/reusable/pretty/avoidable. fuel-p2p labels Jan 31, 2023
@xgreenx xgreenx changed the title add config options for max pending connections, for both outgoing and incoming Add config options for max pending connections, for both outgoing and incoming Jun 14, 2024
@maschad maschad linked a pull request Sep 1, 2024 that will close this issue
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fuel-p2p tech-debt The issue is to improve the current code and make it more clear/generic/reusable/pretty/avoidable.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants