sclgo / impala-go / 12853755844

64%

master: 83%

Build Type

Pull #37

github

Committed by murfffi

Commit Message

fix: root CA support regression

The switch from the deprecated NewTBinaryProtocol to the recommended replacement
NewTBinaryProtocolConf had an unintended side-effect - TLS configuration was
propagated from the protocol to the transport, erasing the previously configured RootCAPool.

This PR fixes that and extends integration tests to cover TLS.

Closes #9

Pull Request Pull Request #37: fix: root CA support regression

Run Details

19 of 26 new or added lines in 1 file covered. (73.08%)

2 existing lines in 1 file now uncovered.

690 of 1080 relevant lines covered (63.89%)

14.14 hits per line