You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kafka.apache.org by "Mickael Maison (Jira)" <ji...@apache.org> on 2022/06/10 11:57:00 UTC

[jira] [Resolved] (KAFKA-13947) Format strings appropriately for their argument types

     [ https://issues.apache.org/jira/browse/KAFKA-13947?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mickael Maison resolved KAFKA-13947.
------------------------------------
    Fix Version/s: 3.3.0
       Resolution: Fixed

> Format strings appropriately for their argument types
> -----------------------------------------------------
>
>                 Key: KAFKA-13947
>                 URL: https://issues.apache.org/jira/browse/KAFKA-13947
>             Project: Kafka
>          Issue Type: Bug
>            Reporter: Divij Vaidya
>            Assignee: Christo Lolov
>            Priority: Minor
>              Labels: n00b, newbee, newbie
>             Fix For: 3.3.0
>
>
> At various places in the code base, String.format() uses '%s' to format long or integer. This is a potential locale-sensitive handling issue.
> The fix should be made at the following places:
> 1. [https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/compress/KafkaLZ4BlockInputStream.java#L176]
> 2. [https://github.com/apache/kafka/blob/trunk/metadata/src/main/java/org/apache/kafka/controller/QuorumController.java#L473]
> 3. [https://github.com/apache/kafka/blob/trunk/raft/src/main/java/org/apache/kafka/raft/KafkaRaftClient.java#L1264]
> 4. [https://github.com/apache/kafka/blob/trunk/raft/src/main/java/org/apache/kafka/raft/ReplicatedCounter.java#L94]
> 5. https://github.com/apache/kafka/blob/trunk/raft/src/main/java/org/apache/kafka/raft/internals/BatchAccumulator.java#L192
> 6. [https://github.com/apache/kafka/blob/trunk/streams/src/main/java/org/apache/kafka/streams/processor/internals/RepartitionTopics.java#L120]
> 7. [https://github.com/apache/kafka/blob/trunk/raft/src/main/java/org/apache/kafka/raft/LeaderState.java#L356]
> 8. https://github.com/apache/kafka/blob/trunk/clients/src/main/java/org/apache/kafka/common/security/oauthbearer/secured/Retry.java#L52
> Note that the files listed above have more such lines where %s is incorrectly used.



--
This message was sent by Atlassian Jira
(v8.20.7#820007)