uber / tchannel-go / 3114

48%

dev: 88%

Pull #585

travis-ci

Only create Connection object for active TChannel connections (#587)

Currently, we create a Connection object as soon as we have a TCP
connection, and then we try to add more information (such as remote peer
information) to the logging context after the init handshake. This can
lead to data races since we're modifying the logging context while
other goroutines may be logging messages with the same context.

Instead of using locks for the logging context, instead separate the
handshake phase of connections out of Connection.

Fixes #584

319 of 319 new or added lines in 7 files covered. (100.0%)

4293 of 4861 relevant lines covered (88.32%)

8954.57 hits per line