support for tracing client including tlsconfig with support for nextprotos

Signed-off-by: David Kydd <david.kydd@microsoft.com>
This commit is contained in:
david kydd 2024-11-04 17:30:11 +13:00
parent 6b36a5592a
commit 3a9895a582
2 changed files with 3 additions and 2 deletions

View file

@ -73,7 +73,7 @@ func (m *Manager) ApplyConfig(cfg *config.Config) error {
// set, we have to restart the manager to make sure that new TLS
// certificates are picked up.
var blankTLSConfig config_util.TLSConfig
if reflect.DeepEqual(m.config, cfg.TracingConfig) && m.config.TLSConfig == blankTLSConfig {
if reflect.DeepEqual(m.config, cfg.TracingConfig) && reflect.DeepEqual(m.config.TLSConfig, blankTLSConfig) {
return nil
}

View file

@ -82,7 +82,8 @@ func TestReinstallingTracerProviderWithTLS(t *testing.T) {
Endpoint: "localhost:1234",
ClientType: config.TracingClientGRPC,
TLSConfig: config_util.TLSConfig{
CAFile: "testdata/ca.cer",
CAFile: "testdata/ca.cer",
NextProtos: []string{"h2"},
},
},
}