[jira] [Commented] (KAFKA-15053) Regression for security.protocol validation starting from 3.3.0
[ https://issues.apache.org/jira/browse/KAFKA-15053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17736529#comment-17736529 ] Chris Egerton commented on KAFKA-15053: --- [~dlgaobo] I think that may have been a typo, but just to be extra clear–we will never backport this to 3.3.0, which has already been released. We may backport it to the 3.3 branch, which will then cause the change to be included if we do another 3.3.x release (right now, that would be 3.3.3). Assuming you're asking about 3.3.x in general and not 3.3.0 specifically, then yes, I believe that once they've upgraded to the new version (e.g., 3.3.3), there's nothing else users will have to do to benefit from this fix. However, again, I'm not certain there will be another 3.3.x release, so they may have to upgrade to a 3.4.x version to benefit from this fix. > Regression for security.protocol validation starting from 3.3.0 > --- > > Key: KAFKA-15053 > URL: https://issues.apache.org/jira/browse/KAFKA-15053 > Project: Kafka > Issue Type: Bug > Components: clients >Affects Versions: 3.3.0 >Reporter: Bo Gao >Assignee: Bo Gao >Priority: Major > > [This|https://issues.apache.org/jira/browse/KAFKA-13793] Jira issue > introduced validations on multiple configs. As a consequence, config > {{security.protocol}} now only allows upper case values such as PLAINTEXT, > SSL, SASL_PLAINTEXT, SASL_SSL. Before this change, lower case values like > sasl_ssl, ssl are also supported, there's even a case insensitive logic > inside > [SecurityProtocol|https://github.com/apache/kafka/blob/146a6976aed0d9f90c70b6f21dca8b887cc34e71/clients/src/main/java/org/apache/kafka/common/security/auth/SecurityProtocol.java#L70-L73] > to handle the lower case values. > I think we should treat this as a regression bug since we don't support lower > case values anymore since 3.3.0. For versions later than 3.3.0, we are > getting error like this when using lower case value sasl_ssl > {{Invalid value sasl_ssl for configuration security.protocol: String must be > one of: PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL}} -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (KAFKA-15053) Regression for security.protocol validation starting from 3.3.0
[ https://issues.apache.org/jira/browse/KAFKA-15053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17736201#comment-17736201 ] Bo Gao commented on KAFKA-15053: Thanks [~ChrisEgerton]! If we backport this to 3.3.0, is there anything that the client needs to do to consume this change or the change would be there automatically? > Regression for security.protocol validation starting from 3.3.0 > --- > > Key: KAFKA-15053 > URL: https://issues.apache.org/jira/browse/KAFKA-15053 > Project: Kafka > Issue Type: Bug > Components: clients >Affects Versions: 3.3.0 >Reporter: Bo Gao >Assignee: Bo Gao >Priority: Major > > [This|https://issues.apache.org/jira/browse/KAFKA-13793] Jira issue > introduced validations on multiple configs. As a consequence, config > {{security.protocol}} now only allows upper case values such as PLAINTEXT, > SSL, SASL_PLAINTEXT, SASL_SSL. Before this change, lower case values like > sasl_ssl, ssl are also supported, there's even a case insensitive logic > inside > [SecurityProtocol|https://github.com/apache/kafka/blob/146a6976aed0d9f90c70b6f21dca8b887cc34e71/clients/src/main/java/org/apache/kafka/common/security/auth/SecurityProtocol.java#L70-L73] > to handle the lower case values. > I think we should treat this as a regression bug since we don't support lower > case values anymore since 3.3.0. For versions later than 3.3.0, we are > getting error like this when using lower case value sasl_ssl > {{Invalid value sasl_ssl for configuration security.protocol: String must be > one of: PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL}} -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (KAFKA-15053) Regression for security.protocol validation starting from 3.3.0
[ https://issues.apache.org/jira/browse/KAFKA-15053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17735933#comment-17735933 ] Bo Gao commented on KAFKA-15053: Hi [~ChrisEgerton] , quick question for the fix of this ticket: can we include it into the 3.3.x minor release as well or we have already stopped doing minor release for 3.3.x? > Regression for security.protocol validation starting from 3.3.0 > --- > > Key: KAFKA-15053 > URL: https://issues.apache.org/jira/browse/KAFKA-15053 > Project: Kafka > Issue Type: Bug > Components: clients >Affects Versions: 3.3.0 >Reporter: Bo Gao >Assignee: Bo Gao >Priority: Major > > [This|https://issues.apache.org/jira/browse/KAFKA-13793] Jira issue > introduced validations on multiple configs. As a consequence, config > {{security.protocol}} now only allows upper case values such as PLAINTEXT, > SSL, SASL_PLAINTEXT, SASL_SSL. Before this change, lower case values like > sasl_ssl, ssl are also supported, there's even a case insensitive logic > inside > [SecurityProtocol|https://github.com/apache/kafka/blob/146a6976aed0d9f90c70b6f21dca8b887cc34e71/clients/src/main/java/org/apache/kafka/common/security/auth/SecurityProtocol.java#L70-L73] > to handle the lower case values. > I think we should treat this as a regression bug since we don't support lower > case values anymore since 3.3.0. For versions later than 3.3.0, we are > getting error like this when using lower case value sasl_ssl > {{Invalid value sasl_ssl for configuration security.protocol: String must be > one of: PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL}} -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (KAFKA-15053) Regression for security.protocol validation starting from 3.3.0
[ https://issues.apache.org/jira/browse/KAFKA-15053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17730767#comment-17730767 ] Bo Gao commented on KAFKA-15053: Hi [~ChrisEgerton] , just created a pull request for the fix [https://github.com/apache/kafka/pull/13831.] I also have a quick question for the release process: imagine this fix is merged and released, to consume this fix, do I need to upgrade to the latest Kafka version? Or is there a way to include this fix in a minor release? Thanks! > Regression for security.protocol validation starting from 3.3.0 > --- > > Key: KAFKA-15053 > URL: https://issues.apache.org/jira/browse/KAFKA-15053 > Project: Kafka > Issue Type: Bug > Components: clients >Affects Versions: 3.3.0 >Reporter: Bo Gao >Priority: Major > > [This|https://issues.apache.org/jira/browse/KAFKA-13793] Jira issue > introduced validations on multiple configs. As a consequence, config > {{security.protocol}} now only allows upper case values such as PLAINTEXT, > SSL, SASL_PLAINTEXT, SASL_SSL. Before this change, lower case values like > sasl_ssl, ssl are also supported, there's even a case insensitive logic > inside > [SecurityProtocol|https://github.com/apache/kafka/blob/146a6976aed0d9f90c70b6f21dca8b887cc34e71/clients/src/main/java/org/apache/kafka/common/security/auth/SecurityProtocol.java#L70-L73] > to handle the lower case values. > I think we should treat this as a regression bug since we don't support lower > case values anymore since 3.3.0. For versions later than 3.3.0, we are > getting error like this when using lower case value sasl_ssl > {{Invalid value sasl_ssl for configuration security.protocol: String must be > one of: PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL}} -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (KAFKA-15053) Regression for security.protocol validation starting from 3.3.0
[ https://issues.apache.org/jira/browse/KAFKA-15053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17729541#comment-17729541 ] Bo Gao commented on KAFKA-15053: Hi [~ChrisEgerton], sure, I can do a fix for that! > Regression for security.protocol validation starting from 3.3.0 > --- > > Key: KAFKA-15053 > URL: https://issues.apache.org/jira/browse/KAFKA-15053 > Project: Kafka > Issue Type: Bug > Components: clients >Affects Versions: 3.3.0 >Reporter: Bo Gao >Priority: Major > > [This|https://issues.apache.org/jira/browse/KAFKA-13793] Jira issue > introduced validations on multiple configs. As a consequence, config > {{security.protocol}} now only allows upper case values such as PLAINTEXT, > SSL, SASL_PLAINTEXT, SASL_SSL. Before this change, lower case values like > sasl_ssl, ssl are also supported, there's even a case insensitive logic > inside > [SecurityProtocol|https://github.com/apache/kafka/blob/146a6976aed0d9f90c70b6f21dca8b887cc34e71/clients/src/main/java/org/apache/kafka/common/security/auth/SecurityProtocol.java#L70-L73] > to handle the lower case values. > I think we should treat this as a regression bug since we don't support lower > case values anymore since 3.3.0. For versions later than 3.3.0, we are > getting error like this when using lower case value sasl_ssl > {{Invalid value sasl_ssl for configuration security.protocol: String must be > one of: PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL}} -- This message was sent by Atlassian Jira (v8.20.10#820010)
[jira] [Commented] (KAFKA-15053) Regression for security.protocol validation starting from 3.3.0
[ https://issues.apache.org/jira/browse/KAFKA-15053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17729323#comment-17729323 ] Chris Egerton commented on KAFKA-15053: --- Thanks [~dlgaobo]. I agree that this is a regression and we should re-introduce case-insensitive support for this this property. Are you interested in providing a patch for this? > Regression for security.protocol validation starting from 3.3.0 > --- > > Key: KAFKA-15053 > URL: https://issues.apache.org/jira/browse/KAFKA-15053 > Project: Kafka > Issue Type: Bug > Components: clients >Affects Versions: 3.3.0 >Reporter: Bo Gao >Priority: Major > > [This|https://issues.apache.org/jira/browse/KAFKA-13793] Jira issue > introduced validations on multiple configs. As a consequence, config > {{security.protocol}} now only allows upper case values such as PLAINTEXT, > SSL, SASL_PLAINTEXT, SASL_SSL. Before this change, lower case values like > sasl_ssl, ssl are also supported, there's even a case insensitive logic > inside > [SecurityProtocol|https://github.com/apache/kafka/blob/146a6976aed0d9f90c70b6f21dca8b887cc34e71/clients/src/main/java/org/apache/kafka/common/security/auth/SecurityProtocol.java#L70-L73] > to handle the lower case values. > I think we should treat this as a regression bug since we don't support lower > case values anymore since 3.3.0. For versions later than 3.3.0, we are > getting error like this when using lower case value sasl_ssl > {{Invalid value sasl_ssl for configuration security.protocol: String must be > one of: PLAINTEXT, SSL, SASL_PLAINTEXT, SASL_SSL}} -- This message was sent by Atlassian Jira (v8.20.10#820010)