You are viewing a plain text version of this content. The canonical link for it is here.

Posted to jira@kafka.apache.org by "bogao007 (via GitHub)" <gi...@apache.org> on 2023/06/10 00:24:09 UTC

[GitHub] [kafka] bogao007 commented on pull request #13831: KAFKA-15053: Use case insensitive validator for security.protocol config

bogao007 commented on PR #13831:
URL: https://github.com/apache/kafka/pull/13831#issuecomment-1585304019

   > For consistency, we should allow case insensitive option at all configuration places for security protocol. e.g. at AdminClientConfig at
   > 
   > https://github.com/apache/kafka/blob/7eea2a3908fdcee1627c18827e6dcb5ed0089fdd/clients/src/main/java/org/apache/kafka/clients/admin/AdminClientConfig.java#L226
   > 
   > and other places like ConsumerConfig, ProducerConfig etc.
   > We should also change the documentation to make it clear that this configuration is case insensitive.
   
   Updated other places by comparing with the [original PR](https://github.com/apache/kafka/pull/12010/files), please help to take another look. Thanks! Also, what's the common process to update the documentation?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jira-unsubscribe@kafka.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org