Chia-Ping Tsai created KAFKA-21269:
--------------------------------------

             Summary: Add tests for describing broker configs with raw string 
values
                 Key: KAFKA-21269
                 URL: https://issues.apache.org/jira/browse/KAFKA-21269
             Project: Kafka
          Issue Type: Bug
            Reporter: Chia-Ping Tsai
            Assignee: Chia-Ping Tsai


{code:java}
    val valueAsString = if (isSensitive)
      null
    else value match {
      case v: String => v
      case _ => ConfigDef.convertToString(value, configEntryType.orNull)
    } {code}
We don’t parse all broker-related values in KafkaConfig. For example, the 
cluster-wide and prefixed configs. Hence, we catch the “raw string” and then 
keep it as it is to avoid casting error.

However, we don’t even have such tests for both cases. #23785 removed the guard 
and CI was still green. We should add integration tests for describing a 
cluster-wide LIST config (e.g. {{{}log.cleanup.policy{}}}) and a prefixed LIST 
config (e.g. {{{}listener.name.external.ssl.enabled.protocols{}}}).



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to