rollback code to set config.net.tls.enable=true when auth is required and tls is not required

Signed-off-by: Sky Ao <aoxiaojian@gmail.com>
This commit is contained in:
Sky Ao 2022-10-13 12:02:05 +00:00 committed by Bernd Verst
parent 5b0679ff8b
commit ec091b8237
1 changed files with 5 additions and 7 deletions

View File

@ -54,15 +54,13 @@ func updateTLSConfig(config *sarama.Config, metadata *kafkaMetadata) error {
config.Net.TLS.Enable = false
return nil
}
if !metadata.TLSSkipVerify && metadata.TLSCaCert == "" {
config.Net.TLS.Enable = false
return nil
}
//nolint:gosec
config.Net.TLS.Config = &tls.Config{InsecureSkipVerify: metadata.TLSSkipVerify, MinVersion: tls.VersionTLS12}
config.Net.TLS.Enable = true
if !metadata.TLSSkipVerify && metadata.TLSCaCert == "" {
return nil
}
//nolint:gosec
config.Net.TLS.Config = &tls.Config{InsecureSkipVerify: metadata.TLSSkipVerify, MinVersion: tls.VersionTLS12}
if metadata.TLSCaCert != "" {
caCertPool := x509.NewCertPool()
if ok := caCertPool.AppendCertsFromPEM([]byte(metadata.TLSCaCert)); !ok {