Allow Netty default leak detection to be overidden

See gh-32144
This commit is contained in:
Russell You 2022-08-23 08:37:42 +10:00 committed by Stephane Nicoll
parent 6d80b601b6
commit 51f2e40c69
3 changed files with 3 additions and 44 deletions

View File

@ -32,7 +32,7 @@ public class NettyProperties {
/**
* Level of leak detection for reference-counted buffers.
*/
private LeakDetection leakDetection = LeakDetection.SIMPLE;
private LeakDetection leakDetection = null;
public LeakDetection getLeakDetection() {
return this.leakDetection;

View File

@ -1738,7 +1738,8 @@
},
{
"name": "spring.netty.leak-detection",
"defaultValue": "simple"
"description": "When null, fall back to Netty system default 'simple' or value of -Dio.netty.leakDetection.level. When set, this will overrride value of -Dio.netty.leakDetection.level",
"defaultValue": "null"
},
{
"name": "spring.quartz.jdbc.comment-prefix",

View File

@ -1,42 +0,0 @@
/*
* Copyright 2012-2021 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.
* You may obtain a copy of the License at
*
* https://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.springframework.boot.autoconfigure.netty;
import io.netty.util.ResourceLeakDetector;
import io.netty.util.ResourceLeakDetector.Level;
import org.junit.jupiter.api.Test;
import org.springframework.test.util.ReflectionTestUtils;
import static org.assertj.core.api.Assertions.assertThat;
/**
* Tests for {@link NettyProperties}
*
* @author Brian Clozel
*/
class NettyPropertiesTests {
@Test
void defaultValueShouldMatchNettys() {
NettyProperties properties = new NettyProperties();
ResourceLeakDetector.Level defaultLevel = (Level) ReflectionTestUtils.getField(ResourceLeakDetector.class,
"DEFAULT_LEVEL");
assertThat(ResourceLeakDetector.Level.valueOf(properties.getLeakDetection().name())).isEqualTo(defaultLevel);
}
}