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

extract endlessServer to support TCP #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

flier
Copy link

@flier flier commented Jul 30, 2015

  1. extract endlessServer to support TCP
  2. improve logging with srv.EndlessListener.Addr()

@fvbock
Copy link
Owner

fvbock commented Aug 3, 2015

thanks a lot for the PR.
had no time to look at it in detail over the weekend - pls give me a little more time to do so before merging. thx...

@flier
Copy link
Author

flier commented Aug 3, 2015

no problem, thanks for your effort :)

Florian von Bock [email protected]于2015年8月3日周一 上午9:35写道:

thanks a lot for the PR.
had no time to look at it in detail over the weekend - pls give me a
little more time to do so before merging. thx...


Reply to this email directly or view it on GitHub
#10 (comment).

@ajpen
Copy link

ajpen commented Mar 4, 2017

Might be better if you factor out the server type and the tcp server into their own files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants