Skip to content

Commit 0bdd1a8

Browse files
Amol VermaAmol Verma
Amol Verma
authored and
Amol Verma
committed
Fix Kafka TLS configuration with plaintext authentication
This change fixes the Kafka TLS configuration to work correctly when using plaintext authentication with TLS enabled. Previously, TLS would only be configured when authentication=tls, breaking SASL-SSL with PLAIN authentication. Resolves jaegertracing#6744 Signed-off-by: Amol Verma <[email protected]>
1 parent 2b355d0 commit 0bdd1a8

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

pkg/kafka/auth/config.go

-1
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ func (config *AuthenticationConfig) InitFromViper(configPrefix string, v *viper.
7676
config.Kerberos.KeyTabPath = v.GetString(configPrefix + kerberosPrefix + suffixKerberosKeyTab)
7777
config.Kerberos.DisablePAFXFast = v.GetBool(configPrefix + kerberosPrefix + suffixKerberosDisablePAFXFAST)
7878

79-
tlsEnabled := v.GetBool(configPrefix + ".tls.enabled")
8079
if config.Authentication == tls || v.GetBool(configPrefix+".tls.enabled") {
8180
tlsClientConfig := tlscfg.ClientFlagsConfig{
8281
Prefix: configPrefix,

0 commit comments

Comments
 (0)