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

Running for a long time causes socket fd leak #42

Closed
NeoChen1024 opened this issue Oct 24, 2023 · 2 comments · Fixed by #21
Closed

Running for a long time causes socket fd leak #42

NeoChen1024 opened this issue Oct 24, 2023 · 2 comments · Fixed by #21
Labels
bug Something isn't working

Comments

@NeoChen1024
Copy link

NeoChen1024 commented Oct 24, 2023

Have you checked the roadmap on the website, and existing issues, before opening a dupllcate issue?
Yes

Describe the bug
After running for a while, the "closed" TCP connection count only goes up, until I manually restart the docker container which bitmagnet is running inside. (could be over 10k after a few hours)
ss -s status

To Reproduce
Steps to reproduce the behavior:

  1. Run it long enough

Expected behavior
"closed" TCP connection count remain low.

General (please complete the following information):

  • Bitmagnet version: 0.0.7
  • OS and version: Arch Linux (kernel 6.1.59)
  • Browser and version (if issue is with WebUI): Firefox 118.02

Additional context
None

@NeoChen1024 NeoChen1024 added the bug Something isn't working label Oct 24, 2023
@NeoChen1024
Copy link
Author

After left running for a night:
screenshot of ss -s

@mgdigital
Copy link
Collaborator

Hi @NeoChen1024 , I believe this is fixed in the linked PR which will be merged shortly.

mgdigital added a commit that referenced this issue Nov 6, 2023
- Address #11 with BEP5 & BEP51 implementations
- Reimplement DHT crawler around new routing table
- Fix #45
- Fix #42
- Fix #39
- Fix #37
- Make bootstrap nodes configurable: #27
- Add profiling tools to Docker image
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants