dino2895 opened a new pull request, #22872: URL: https://github.com/apache/kafka/pull/22872
This follow-up addresses review feedback from #20376. The client security configuration was previously split between `setClientSaslConfig` and `setClientSslConfig`, resulting in nested and counterintuitive call chains. This change: - Introduces `createClientSecurityConfig` as the unified client security configuration API. - Centralizes PLAINTEXT, SSL, SASL_PLAINTEXT, and SASL_SSL handling in `RaftClusterInstance`. - Updates producer, consumer, share consumer, and admin client creation. - Updates the direct client integration test caller. - Removes the obsolete SASL- and SSL-specific methods from `ClusterInstance`. Testing: - `./gradlew :test-common:test-common-runtime:spotlessCheck` - `./gradlew :test-common:test-common-runtime:checkstyleMain` - `./gradlew :test-common:test-common-runtime:compileJava` - `./gradlew :test-common:test-common-runtime:test --tests org.apache.kafka.common.test.junit.ClusterTestExtensionsTest --rerun-tasks` - `./gradlew :clients:clients-integration-tests:test --tests org.apache.kafka.clients.admin.ConcurrentListOffsetsRequestTest` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
