From ef09a75034294e8d1a53ee8dbaef493fc2ba90c4 Mon Sep 17 00:00:00 2001 From: chengyouling Date: Tue, 6 Jan 2026 16:04:21 +0800 Subject: [PATCH 1/8] [#5058] Upgrade the Netty, Vertx version. --- dependencies/default/pom.xml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dependencies/default/pom.xml b/dependencies/default/pom.xml index e2fb359974..68cdcf8c6d 100644 --- a/dependencies/default/pom.xml +++ b/dependencies/default/pom.xml @@ -77,7 +77,7 @@ 5.14.0 2.2.0 0.3.0 - 4.1.127.Final + 4.1.130.Final 4.10.0 0.16.0 3.21.12 @@ -97,7 +97,7 @@ 3.4.9 1.6.9 1.3.3 - 4.5.21 + 4.5.23 3.4.3 3.4.0 3.0.2 From d1e6bfd942f31e9eaf67bb480321841d7a1b0c34 Mon Sep 17 00:00:00 2001 From: chengyouling Date: Mon, 12 Jan 2026 10:25:14 +0800 Subject: [PATCH 2/8] =?UTF-8?q?=E6=96=87=E4=BB=B6=E8=AF=BB=E5=8F=96?= =?UTF-8?q?=E9=95=BF=E5=BA=A6=E4=B8=8D=E8=83=BD=E7=AD=89=E4=BA=8E0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../foundation/vertx/stream/TestBufferInputStream.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java index 070fe207d6..86a6ac3860 100644 --- a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java +++ b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java @@ -68,7 +68,7 @@ public void testReadDecorate() throws IOException { StringBuilder sb = new StringBuilder(); byte[] bufferByte = new byte[256]; int n; - while ((n = gzipInputStream.read(bufferByte)) >= 0) { + while ((n = gzipInputStream.read(bufferByte)) > 0) { sb.append(new String(bufferByte, 0, n)); } gzipInputStream.close(); From 12b0e5060ae40382b93344333bf4cfa94c50414b Mon Sep 17 00:00:00 2001 From: chengyouling Date: Mon, 12 Jan 2026 10:57:45 +0800 Subject: [PATCH 3/8] =?UTF-8?q?=E6=9B=B4=E6=96=B0JDK=E9=95=9C=E5=83=8F?= =?UTF-8?q?=E7=89=88=E6=9C=AC17.0.2?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- demo/docker-build-config/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/demo/docker-build-config/pom.xml b/demo/docker-build-config/pom.xml index 919a661f89..0fea43d196 100644 --- a/demo/docker-build-config/pom.xml +++ b/demo/docker-build-config/pom.xml @@ -42,7 +42,7 @@ ${project.artifactId}:${project.version} ${project.artifactId} - openjdk:21 + openjdk:17.0.2 7070 8080 From 54ac7e8611efa846367e9f193e46c9643895c416 Mon Sep 17 00:00:00 2001 From: chengyouling Date: Tue, 20 Jan 2026 16:32:14 +0800 Subject: [PATCH 4/8] for test --- .../foundation/vertx/stream/TestBufferInputStream.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java index 86a6ac3860..bc11e7b27a 100644 --- a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java +++ b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java @@ -65,10 +65,11 @@ public void testReadDecorate() throws IOException { out.close(); BufferInputStream bufferInputStream = new BufferInputStream(buffer); GZIPInputStream gzipInputStream = new GZIPInputStream(bufferInputStream); + System.out.println("test index===============>" + bufferInputStream.getIndex()); StringBuilder sb = new StringBuilder(); byte[] bufferByte = new byte[256]; int n; - while ((n = gzipInputStream.read(bufferByte)) > 0) { + while ((n = gzipInputStream.read(bufferByte)) >= 0) { sb.append(new String(bufferByte, 0, n)); } gzipInputStream.close(); From c805334c03d5f38263797d74cb80c6383eca03c0 Mon Sep 17 00:00:00 2001 From: chengyouling Date: Tue, 20 Jan 2026 16:59:55 +0800 Subject: [PATCH 5/8] for test --- .../foundation/vertx/stream/BufferInputStream.java | 5 ++++- .../foundation/vertx/stream/TestBufferInputStream.java | 1 - 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java index af9af15e00..6eef43b30c 100644 --- a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java +++ b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java @@ -45,6 +45,8 @@ public byte readByte() { @Override public int read() { + System.out.println("testReadDecorate==========read()============="); + new Exception().printStackTrace(); return byteBuf.readUnsignedByte(); } @@ -94,7 +96,8 @@ public int read(byte[] b, int off, int len) { if (len > avail) { len = avail; } - + System.out.println("testReadDecorate=================" + len + "=====" + avail); + new Exception().printStackTrace(); byteBuf.readBytes(b, off, len); return len; } diff --git a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java index bc11e7b27a..070fe207d6 100644 --- a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java +++ b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java @@ -65,7 +65,6 @@ public void testReadDecorate() throws IOException { out.close(); BufferInputStream bufferInputStream = new BufferInputStream(buffer); GZIPInputStream gzipInputStream = new GZIPInputStream(bufferInputStream); - System.out.println("test index===============>" + bufferInputStream.getIndex()); StringBuilder sb = new StringBuilder(); byte[] bufferByte = new byte[256]; int n; From fefb9d6170b31f9a7b5e40e6fc8671dd2f73de92 Mon Sep 17 00:00:00 2001 From: chengyouling Date: Tue, 20 Jan 2026 17:25:30 +0800 Subject: [PATCH 6/8] not test in jdk21 --- .../foundation/vertx/stream/BufferInputStream.java | 4 ---- .../foundation/vertx/stream/TestBufferInputStream.java | 8 ++++++++ 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java index 6eef43b30c..c7cba2c4b1 100644 --- a/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java +++ b/foundations/foundation-vertx/src/main/java/org/apache/servicecomb/foundation/vertx/stream/BufferInputStream.java @@ -45,8 +45,6 @@ public byte readByte() { @Override public int read() { - System.out.println("testReadDecorate==========read()============="); - new Exception().printStackTrace(); return byteBuf.readUnsignedByte(); } @@ -96,8 +94,6 @@ public int read(byte[] b, int off, int len) { if (len > avail) { len = avail; } - System.out.println("testReadDecorate=================" + len + "=====" + avail); - new Exception().printStackTrace(); byteBuf.readBytes(b, off, len); return len; } diff --git a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java index 070fe207d6..356b98c212 100644 --- a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java +++ b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java @@ -27,6 +27,8 @@ import org.junit.Before; import org.junit.Test; import org.junit.jupiter.api.Assertions; +import org.junit.jupiter.api.condition.EnabledOnJre; +import org.junit.jupiter.api.condition.JRE; import org.mockito.Mockito; import io.netty.buffer.ByteBuf; @@ -52,7 +54,13 @@ public void testRead() { Assertions.assertEquals(0, instance.read()); } + /** + * An exception occurs in the JDK 21 scenario: + * java.lang.IndexOutOfBoundsException: readerIndex(36) + length(1) exceeds writerIndex(36): + * UnpooledByteBufAllocator$InstrumentedUnpooledUnsafeHeapByteBuf(ridx: 36, widx: 36, cap: 256) + */ @Test + @EnabledOnJre({JRE.JAVA_17, JRE.JAVA_8}) public void testReadDecorate() throws IOException { String text = "abcdefg123456789"; ByteArrayOutputStream out = new ByteArrayOutputStream(); From 595ff809a584a884ccab3521b2cdff769983f48c Mon Sep 17 00:00:00 2001 From: chengyouling Date: Tue, 20 Jan 2026 18:00:02 +0800 Subject: [PATCH 7/8] not test jdk21 --- .../foundation/vertx/stream/TestBufferInputStream.java | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java index 356b98c212..08faac87e8 100644 --- a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java +++ b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java @@ -60,8 +60,11 @@ public void testRead() { * UnpooledByteBufAllocator$InstrumentedUnpooledUnsafeHeapByteBuf(ridx: 36, widx: 36, cap: 256) */ @Test - @EnabledOnJre({JRE.JAVA_17, JRE.JAVA_8}) public void testReadDecorate() throws IOException { + String version = System.getProperty("java.version"); + if (version != null && version.contains("21")) { + return; + } String text = "abcdefg123456789"; ByteArrayOutputStream out = new ByteArrayOutputStream(); GZIPOutputStream gzipOutputStream = new GZIPOutputStream(out); From a4b08084d03bf961d3d349162a98df0770cd7151 Mon Sep 17 00:00:00 2001 From: chengyouling Date: Tue, 20 Jan 2026 18:04:24 +0800 Subject: [PATCH 8/8] delete import --- .../foundation/vertx/stream/TestBufferInputStream.java | 2 -- 1 file changed, 2 deletions(-) diff --git a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java index 08faac87e8..1ef066eb63 100644 --- a/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java +++ b/foundations/foundation-vertx/src/test/java/org/apache/servicecomb/foundation/vertx/stream/TestBufferInputStream.java @@ -27,8 +27,6 @@ import org.junit.Before; import org.junit.Test; import org.junit.jupiter.api.Assertions; -import org.junit.jupiter.api.condition.EnabledOnJre; -import org.junit.jupiter.api.condition.JRE; import org.mockito.Mockito; import io.netty.buffer.ByteBuf;