Merge branch '2.6.x' into 2.7.x

Closes gh-32145
spring-projects · Aug 23, 2022 · 7a91fa9 · 7a91fa9
2 parents 392aec4 + 463766f
commit 7a91fa9
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 11 deletions.
diff --git a/...configure/src/main/java/org/springframework/boot/autoconfigure/netty/NettyProperties.java b/...configure/src/main/java/org/springframework/boot/autoconfigure/netty/NettyProperties.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2021 the original author or authors.
+ * Copyright 2012-2022 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -30,9 +30,11 @@
 public class NettyProperties {
 
 	/**
-	 * Level of leak detection for reference-counted buffers.
+	 * Level of leak detection for reference-counted buffers. If not configured via
+	 * 'ResourceLeakDetector.setLevel' or the 'io.netty.leakDetection.level' system
+	 * property, default to 'simple'.
 	 */
-	private LeakDetection leakDetection = LeakDetection.SIMPLE;
+	private LeakDetection leakDetection;
 
 	public LeakDetection getLeakDetection() {
 		return this.leakDetection;

diff --git a/...t-autoconfigure/src/main/resources/META-INF/additional-spring-configuration-metadata.json b/...t-autoconfigure/src/main/resources/META-INF/additional-spring-configuration-metadata.json
@@ -1862,10 +1862,6 @@
       "name": "spring.neo4j.uri",
       "defaultValue": "bolt://localhost:7687"
     },
-    {
-      "name": "spring.netty.leak-detection",
-      "defaultValue": "simple"
-    },
     {
       "name": "spring.quartz.jdbc.comment-prefix",
       "defaultValue": [

diff --git a/...gure/src/test/java/org/springframework/boot/autoconfigure/netty/NettyPropertiesTests.java b/...gure/src/test/java/org/springframework/boot/autoconfigure/netty/NettyPropertiesTests.java
@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2021 the original author or authors.
+ * Copyright 2012-2022 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -32,11 +32,10 @@
 class NettyPropertiesTests {
 
 	@Test
-	void defaultValueShouldMatchNettys() {
-		NettyProperties properties = new NettyProperties();
+	void defaultValueShouldBeConsistent() {
 		ResourceLeakDetector.Level defaultLevel = (Level) ReflectionTestUtils.getField(ResourceLeakDetector.class,
 				"DEFAULT_LEVEL");
-		assertThat(ResourceLeakDetector.Level.valueOf(properties.getLeakDetection().name())).isEqualTo(defaultLevel);
+		assertThat(defaultLevel).isEqualTo(Level.SIMPLE);
 	}
 
 }