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.

Sign up for GitHub

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

Jump to bottom

feat: change "connected clients" logging as DEBUG #16

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

feat: change "connected clients" logging as DEBUG #16

wants to merge 9 commits into from

Conversation

dhde
Copy link

@dhde dhde commented Jan 5, 2022

No description provided.

@dhde dhde mentioned this pull request Jan 5, 2022
Open
@jcmoraisjr
Copy link
Owner

I've just rebased your own commit 5990cd2, which implements only the annoying "xx clients connected" message every second. Do you all say that the others, removed in ce58873, would also be useful to be removed? some of them relates to connection issues and would be useful to see it there. Thoughts?

@bguscott
Copy link

bguscott commented Apr 7, 2023

It would be useful to also remove/change "Peer closed connection" to debug as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dhde @jcmoraisjr @bguscott