Skip to content

Commit 8ae8ae4

Browse files
authored
CVE fixes, July 23 (#4003)
1 parent 5f231c7 commit 8ae8ae4

File tree

5 files changed

+32
-46
lines changed

5 files changed

+32
-46
lines changed

kafka-ui-api/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@
9191
<dependency>
9292
<groupId>software.amazon.msk</groupId>
9393
<artifactId>aws-msk-iam-auth</artifactId>
94-
<version>1.1.6</version>
94+
<version>1.1.7</version>
9595
</dependency>
9696

9797
<dependency>

kafka-ui-api/src/test/java/com/provectus/kafka/ui/AbstractIntegrationTest.java

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.provectus.kafka.ui;
22

33
import com.provectus.kafka.ui.container.KafkaConnectContainer;
4+
import com.provectus.kafka.ui.container.KsqlDbContainer;
45
import com.provectus.kafka.ui.container.SchemaRegistryContainer;
56
import java.nio.file.Path;
67
import java.util.List;
@@ -32,7 +33,7 @@ public abstract class AbstractIntegrationTest {
3233
public static final String LOCAL = "local";
3334
public static final String SECOND_LOCAL = "secondLocal";
3435

35-
private static final String CONFLUENT_PLATFORM_VERSION = "5.5.0";
36+
private static final String CONFLUENT_PLATFORM_VERSION = "7.2.1"; // Append ".arm64" for a local run
3637

3738
public static final KafkaContainer kafka = new KafkaContainer(
3839
DockerImageName.parse("confluentinc/cp-kafka").withTag(CONFLUENT_PLATFORM_VERSION))
@@ -49,6 +50,11 @@ public abstract class AbstractIntegrationTest {
4950
.dependsOn(kafka)
5051
.dependsOn(schemaRegistry);
5152

53+
protected static final KsqlDbContainer KSQL_DB = new KsqlDbContainer(
54+
DockerImageName.parse("confluentinc/cp-ksqldb-server")
55+
.withTag(CONFLUENT_PLATFORM_VERSION))
56+
.withKafka(kafka);
57+
5258
@TempDir
5359
public static Path tmpDir;
5460

kafka-ui-api/src/test/java/com/provectus/kafka/ui/service/ksql/KsqlApiClientTest.java

Lines changed: 21 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -3,28 +3,22 @@
33
import static org.assertj.core.api.Assertions.assertThat;
44

55
import com.fasterxml.jackson.databind.node.ArrayNode;
6-
import com.fasterxml.jackson.databind.node.DoubleNode;
6+
import com.fasterxml.jackson.databind.node.DecimalNode;
77
import com.fasterxml.jackson.databind.node.IntNode;
88
import com.fasterxml.jackson.databind.node.JsonNodeFactory;
99
import com.fasterxml.jackson.databind.node.TextNode;
1010
import com.provectus.kafka.ui.AbstractIntegrationTest;
11-
import com.provectus.kafka.ui.container.KsqlDbContainer;
11+
import java.math.BigDecimal;
1212
import java.time.Duration;
13-
import java.util.List;
1413
import java.util.Map;
1514
import org.junit.jupiter.api.AfterAll;
1615
import org.junit.jupiter.api.BeforeAll;
1716
import org.junit.jupiter.api.Test;
1817
import org.testcontainers.shaded.org.awaitility.Awaitility;
19-
import org.testcontainers.utility.DockerImageName;
2018
import reactor.test.StepVerifier;
2119

2220
class KsqlApiClientTest extends AbstractIntegrationTest {
2321

24-
private static final KsqlDbContainer KSQL_DB = new KsqlDbContainer(
25-
DockerImageName.parse("confluentinc/ksqldb-server").withTag("0.24.0"))
26-
.withKafka(kafka);
27-
2822
@BeforeAll
2923
static void startContainer() {
3024
KSQL_DB.start();
@@ -72,7 +66,7 @@ void ksqTutorialQueriesWork() {
7266
private void assertLastKsqTutorialQueryResult(KsqlApiClient client) {
7367
// expected results:
7468
//{"header":"Schema","columnNames":[...],"values":null}
75-
//{"header":"Row","columnNames":null,"values":[[0.0,["4ab5cbad","8b6eae59","4a7c7b41"],3]]}
69+
//{"header":"Row","columnNames":null,"values":[[0,["4ab5cbad","8b6eae59","4a7c7b41"],3]]}
7670
//{"header":"Row","columnNames":null,"values":[[10.0,["18f4ea86"],1]]}
7771
StepVerifier.create(
7872
client.execute(
@@ -86,34 +80,26 @@ private void assertLastKsqTutorialQueryResult(KsqlApiClient client) {
8680
assertThat(header.getValues()).isNull();
8781
})
8882
.assertNext(row -> {
89-
assertThat(row).isEqualTo(
90-
KsqlApiClient.KsqlResponseTable.builder()
91-
.header("Row")
92-
.columnNames(null)
93-
.values(List.of(List.of(
94-
new DoubleNode(0.0),
95-
new ArrayNode(JsonNodeFactory.instance)
96-
.add(new TextNode("4ab5cbad"))
97-
.add(new TextNode("8b6eae59"))
98-
.add(new TextNode("4a7c7b41")),
99-
new IntNode(3)
100-
)))
101-
.build()
102-
);
83+
var distance = (DecimalNode) row.getValues().get(0).get(0);
84+
var riders = (ArrayNode) row.getValues().get(0).get(1);
85+
var count = (IntNode) row.getValues().get(0).get(2);
86+
87+
assertThat(distance).isEqualTo(new DecimalNode(new BigDecimal(0)));
88+
assertThat(riders).isEqualTo(new ArrayNode(JsonNodeFactory.instance)
89+
.add(new TextNode("4ab5cbad"))
90+
.add(new TextNode("8b6eae59"))
91+
.add(new TextNode("4a7c7b41")));
92+
assertThat(count).isEqualTo(new IntNode(3));
10393
})
10494
.assertNext(row -> {
105-
assertThat(row).isEqualTo(
106-
KsqlApiClient.KsqlResponseTable.builder()
107-
.header("Row")
108-
.columnNames(null)
109-
.values(List.of(List.of(
110-
new DoubleNode(10.0),
111-
new ArrayNode(JsonNodeFactory.instance)
112-
.add(new TextNode("18f4ea86")),
113-
new IntNode(1)
114-
)))
115-
.build()
116-
);
95+
var distance = (DecimalNode) row.getValues().get(0).get(0);
96+
var riders = (ArrayNode) row.getValues().get(0).get(1);
97+
var count = (IntNode) row.getValues().get(0).get(2);
98+
99+
assertThat(distance).isEqualTo(new DecimalNode(new BigDecimal(10)));
100+
assertThat(riders).isEqualTo(new ArrayNode(JsonNodeFactory.instance)
101+
.add(new TextNode("18f4ea86")));
102+
assertThat(count).isEqualTo(new IntNode(1));
117103
})
118104
.verifyComplete();
119105
}

kafka-ui-api/src/test/java/com/provectus/kafka/ui/service/ksql/KsqlServiceV2Test.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@
33
import static org.assertj.core.api.Assertions.assertThat;
44

55
import com.provectus.kafka.ui.AbstractIntegrationTest;
6-
import com.provectus.kafka.ui.container.KsqlDbContainer;
76
import com.provectus.kafka.ui.model.KafkaCluster;
87
import com.provectus.kafka.ui.model.KsqlStreamDescriptionDTO;
98
import com.provectus.kafka.ui.model.KsqlTableDescriptionDTO;
@@ -15,14 +14,9 @@
1514
import org.junit.jupiter.api.AfterAll;
1615
import org.junit.jupiter.api.BeforeAll;
1716
import org.junit.jupiter.api.Test;
18-
import org.testcontainers.utility.DockerImageName;
1917

2018
class KsqlServiceV2Test extends AbstractIntegrationTest {
2119

22-
private static final KsqlDbContainer KSQL_DB = new KsqlDbContainer(
23-
DockerImageName.parse("confluentinc/ksqldb-server").withTag("0.24.0"))
24-
.withKafka(kafka);
25-
2620
private static final Set<String> STREAMS_TO_DELETE = new CopyOnWriteArraySet<>();
2721
private static final Set<String> TABLES_TO_DELETE = new CopyOnWriteArraySet<>();
2822

pom.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,17 +26,17 @@
2626
<assertj.version>3.19.0</assertj.version>
2727
<avro.version>1.11.1</avro.version>
2828
<byte-buddy.version>1.12.19</byte-buddy.version>
29-
<confluent.version>7.3.3</confluent.version>
29+
<confluent.version>7.4.0</confluent.version>
3030
<datasketches-java.version>3.1.0</datasketches-java.version>
3131
<groovy.version>3.0.13</groovy.version>
3232
<jackson.version>2.14.0</jackson.version>
33-
<kafka-clients.version>3.3.1</kafka-clients.version>
33+
<kafka-clients.version>3.5.0</kafka-clients.version>
3434
<org.mapstruct.version>1.5.5.Final</org.mapstruct.version>
3535
<org.projectlombok.version>1.18.24</org.projectlombok.version>
3636
<protobuf-java.version>3.23.3</protobuf-java.version>
3737
<scala-lang.library.version>2.13.9</scala-lang.library.version>
3838
<snakeyaml.version>2.0</snakeyaml.version>
39-
<spring-boot.version>3.0.6</spring-boot.version>
39+
<spring-boot.version>3.1.1</spring-boot.version>
4040
<kafka-ui-serde-api.version>1.0.0</kafka-ui-serde-api.version>
4141
<odd-oddrn-generator.version>0.1.17</odd-oddrn-generator.version>
4242
<odd-oddrn-client.version>0.1.23</odd-oddrn-client.version>

0 commit comments

Comments
 (0)