tree 8edc3a6eb3ff75b1eab7809429ab12aa39d59c2c
parent d9fcdd3dbafbe1a8296018d0d6c55d972f607a42
author Yuxuan 'fishy' Wang <yuxuan.wang@reddit.com> 1611358901 -0800
committer Yuxuan 'fishy' Wang <fishywang@gmail.com> 1611377397 -0800

THRIFT-5322: THeaderTransport protocol id fix

Client: go

This fixes a bug introduced in
https://github.com/apache/thrift/pull/2296, that we mixed the preferred
proto id and the detected proto id, which was a bad idea.

This change separates them, so when we propagate TConfiguration, we only
change the preferred one, which will only be used for new connections,
and leave the detected one from existing connections untouched.

Also add a test for it.