KAFKA-14962: Trim whitespace from ACL configuration

[divijvaidya]

Kafka's startup can fail (see the exception trace below) if there is a trailing or leading whitespace for a configuration value. This fix makes it more tolerant towards cases where a user might accidentally add a trailing or leading whitespace in ACL configuration.

ERROR [KafkaServer id=3] Fatal error during KafkaServer startup. Prepare to shutdown (kafka.server.KafkaServer)

java.lang.IllegalArgumentException: For input string: "true "

    at scala.collection.StringOps$.toBooleanImpl$extension(StringOps.scala:943)

    at kafka.security.authorizer.AclAuthorizer.$anonfun$configure$4(AclAuthorizer.scala:153) 

[divijvaidya]

The failing tests are unrelated:

Build / JDK 8 and Scala 2.12 / testElectUncleanLeadersForOnePartition(String).quorum=kraft – kafka.api.PlaintextAdminIntegrationTest
26s
Build / JDK 11 and Scala 2.13 / testSingleNodeCluster() – org.apache.kafka.connect.mirror.integration.DedicatedMirrorIntegrationTest
1m 18s
Build / JDK 11 and Scala 2.13 / testMultiNodeCluster() – org.apache.kafka.connect.mirror.integration.DedicatedMirrorIntegrationTest
1m 52s
Build / JDK 11 and Scala 2.13 / testReplicateSourceDefault() – org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationExactlyOnceTest

[clolov]

I believe I have encountered similar problems even with configurations so it might be useful to circle back later on and check whether trimming happens there. Thanks!

[omkreddy]

@divijvaidya Thanks for the PR. LGTM

[divijvaidya]

@omkreddy Thank you for your review bandwidth to merge this in 🙏