Skip to content

Wrap TCPListener with keep-alive + timeout, like it's done in net/http - #44

Open
aviddiviner wants to merge 1 commit into
braintree:masterfrom
aviddiviner:keepalive
Open

Wrap TCPListener with keep-alive + timeout, like it's done in net/http#44
aviddiviner wants to merge 1 commit into
braintree:masterfrom
aviddiviner:keepalive

Conversation

@aviddiviner

Copy link
Copy Markdown

What do you think of this?

I took it from here in net/http (tcpKeepAliveListener). They wrap their net.Listener in order to conn.SetKeepAlive(true) on each accepted connection when using http.ListenAndServe.

As you can see from their docs (emphasis mine):

ListenAndServe listens on the TCP network address srv.Addr and then calls Serve to handle requests on incoming connections. Accepted connections are configured to enable TCP keep-alives.

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.

1 participant