ringpop-go icon indicating copy to clipboard operation
ringpop-go copied to clipboard

Support all (3) protocols of TChannel in HandleOrForward

Open thanodnl opened this issue 9 years ago • 0 comments

Currently thrift over TChannel is not working as reported by https://github.com/uber/ringpop-go/pull/22. Before JSON over TChannel was not working as fixed by https://github.com/uber/ringpop-go/pull/14.

As one could see is that both PR's are the exact inverse of each other.

Since HandleOrForward is one of -maybe the biggest- driver to use Ringpop it should work on all supported TChannel protocols. We need to figure out how to do this the correct way.

thanodnl avatar Oct 20 '15 09:10 thanodnl