Merge branch '3.0.x' into 3.1.x

Closes gh-35922
pull/36038/head
Phillip Webb 1 year ago
commit c192fb85b2

@ -1,5 +1,5 @@
/* /*
* Copyright 2012-2022 the original author or authors. * Copyright 2012-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -34,6 +34,8 @@ import org.springframework.boot.actuate.logging.LoggersEndpoint.GroupLoggerLevel
import org.springframework.boot.actuate.logging.LoggersEndpoint.SingleLoggerLevelsDescriptor; import org.springframework.boot.actuate.logging.LoggersEndpoint.SingleLoggerLevelsDescriptor;
import org.springframework.boot.logging.LogLevel; import org.springframework.boot.logging.LogLevel;
import org.springframework.boot.logging.LoggerConfiguration; import org.springframework.boot.logging.LoggerConfiguration;
import org.springframework.boot.logging.LoggerConfiguration.ConfigurationScope;
import org.springframework.boot.logging.LoggerConfiguration.LevelConfiguration;
import org.springframework.boot.logging.LoggerGroup; import org.springframework.boot.logging.LoggerGroup;
import org.springframework.boot.logging.LoggerGroups; import org.springframework.boot.logging.LoggerGroups;
import org.springframework.boot.logging.LoggingSystem; import org.springframework.boot.logging.LoggingSystem;
@ -164,7 +166,11 @@ public class LoggersEndpoint {
private final String configuredLevel; private final String configuredLevel;
public LoggerLevelsDescriptor(LogLevel configuredLevel) { public LoggerLevelsDescriptor(LogLevel configuredLevel) {
this.configuredLevel = getName(configuredLevel); this.configuredLevel = (configuredLevel != null) ? configuredLevel.name() : null;
}
LoggerLevelsDescriptor(LevelConfiguration directConfiguration) {
this.configuredLevel = (directConfiguration != null) ? directConfiguration.getName() : null;
} }
protected final String getName(LogLevel level) { protected final String getName(LogLevel level) {
@ -203,8 +209,8 @@ public class LoggersEndpoint {
private final String effectiveLevel; private final String effectiveLevel;
public SingleLoggerLevelsDescriptor(LoggerConfiguration configuration) { public SingleLoggerLevelsDescriptor(LoggerConfiguration configuration) {
super(configuration.getConfiguredLevel()); super(configuration.getLevelConfiguration(ConfigurationScope.DIRECT));
this.effectiveLevel = getName(configuration.getEffectiveLevel()); this.effectiveLevel = configuration.getLevelConfiguration().getName();
} }
public String getEffectiveLevel() { public String getEffectiveLevel() {

@ -35,6 +35,7 @@ import org.springframework.boot.actuate.logging.LoggersEndpoint.LoggersDescripto
import org.springframework.boot.actuate.logging.LoggersEndpoint.SingleLoggerLevelsDescriptor; import org.springframework.boot.actuate.logging.LoggersEndpoint.SingleLoggerLevelsDescriptor;
import org.springframework.boot.logging.LogLevel; import org.springframework.boot.logging.LogLevel;
import org.springframework.boot.logging.LoggerConfiguration; import org.springframework.boot.logging.LoggerConfiguration;
import org.springframework.boot.logging.LoggerConfiguration.LevelConfiguration;
import org.springframework.boot.logging.LoggerGroups; import org.springframework.boot.logging.LoggerGroups;
import org.springframework.boot.logging.LoggingSystem; import org.springframework.boot.logging.LoggingSystem;
@ -113,6 +114,17 @@ class LoggersEndpointTests {
assertThat(levels.getEffectiveLevel()).isEqualTo("DEBUG"); assertThat(levels.getEffectiveLevel()).isEqualTo("DEBUG");
} }
@Test // gh-35227
void loggerLevelsWhenCustomLevelShouldReturnLevels() {
given(this.loggingSystem.getLoggerConfiguration("ROOT"))
.willReturn(new LoggerConfiguration("ROOT", null, LevelConfiguration.ofCustom("FINEST")));
SingleLoggerLevelsDescriptor levels = (SingleLoggerLevelsDescriptor) new LoggersEndpoint(this.loggingSystem,
this.loggerGroups)
.loggerLevels("ROOT");
assertThat(levels.getConfiguredLevel()).isNull();
assertThat(levels.getEffectiveLevel()).isEqualTo("FINEST");
}
@Test @Test
void groupNameSpecifiedShouldReturnConfiguredLevelAndMembers() { void groupNameSpecifiedShouldReturnConfiguredLevelAndMembers() {
GroupLoggerLevelsDescriptor levels = (GroupLoggerLevelsDescriptor) new LoggersEndpoint(this.loggingSystem, GroupLoggerLevelsDescriptor levels = (GroupLoggerLevelsDescriptor) new LoggersEndpoint(this.loggingSystem,

@ -1,5 +1,5 @@
/* /*
* Copyright 2012-2022 the original author or authors. * Copyright 2012-2023 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
@ -16,6 +16,8 @@
package org.springframework.boot.logging; package org.springframework.boot.logging;
import java.util.Objects;
import org.springframework.util.Assert; import org.springframework.util.Assert;
import org.springframework.util.ObjectUtils; import org.springframework.util.ObjectUtils;
@ -23,15 +25,16 @@ import org.springframework.util.ObjectUtils;
* Immutable class that represents the configuration of a {@link LoggingSystem}'s logger. * Immutable class that represents the configuration of a {@link LoggingSystem}'s logger.
* *
* @author Ben Hale * @author Ben Hale
* @author Phillip Webb
* @since 1.5.0 * @since 1.5.0
*/ */
public final class LoggerConfiguration { public final class LoggerConfiguration {
private final String name; private final String name;
private final LogLevel configuredLevel; private final LevelConfiguration levelConfiguration;
private final LogLevel effectiveLevel; private final LevelConfiguration inheritedLevelConfiguration;
/** /**
* Create a new {@link LoggerConfiguration instance}. * Create a new {@link LoggerConfiguration instance}.
@ -43,32 +46,72 @@ public final class LoggerConfiguration {
Assert.notNull(name, "Name must not be null"); Assert.notNull(name, "Name must not be null");
Assert.notNull(effectiveLevel, "EffectiveLevel must not be null"); Assert.notNull(effectiveLevel, "EffectiveLevel must not be null");
this.name = name; this.name = name;
this.configuredLevel = configuredLevel; this.levelConfiguration = (configuredLevel != null) ? LevelConfiguration.of(configuredLevel) : null;
this.effectiveLevel = effectiveLevel; this.inheritedLevelConfiguration = LevelConfiguration.of(effectiveLevel);
}
/**
* Create a new {@link LoggerConfiguration instance}.
* @param name the name of the logger
* @param levelConfiguration the level configuration
* @param inheritedLevelConfiguration the inherited level configuration
* @since 2.7.13
*/
public LoggerConfiguration(String name, LevelConfiguration levelConfiguration,
LevelConfiguration inheritedLevelConfiguration) {
Assert.notNull(name, "Name must not be null");
Assert.notNull(inheritedLevelConfiguration, "EffectiveLevelConfiguration must not be null");
this.name = name;
this.levelConfiguration = levelConfiguration;
this.inheritedLevelConfiguration = inheritedLevelConfiguration;
}
/**
* Returns the name of the logger.
* @return the name of the logger
*/
public String getName() {
return this.name;
} }
/** /**
* Returns the configured level of the logger. * Returns the configured level of the logger.
* @return the configured level of the logger * @return the configured level of the logger
* @see #getLevelConfiguration(ConfigurationScope)
*/ */
public LogLevel getConfiguredLevel() { public LogLevel getConfiguredLevel() {
return this.configuredLevel; LevelConfiguration configuration = getLevelConfiguration(ConfigurationScope.DIRECT);
return (configuration != null) ? configuration.getLevel() : null;
} }
/** /**
* Returns the effective level of the logger. * Returns the effective level of the logger.
* @return the effective level of the logger * @return the effective level of the logger
* @see #getLevelConfiguration(ConfigurationScope)
*/ */
public LogLevel getEffectiveLevel() { public LogLevel getEffectiveLevel() {
return this.effectiveLevel; return getLevelConfiguration().getLevel();
} }
/** /**
* Returns the name of the logger. * Return the level configuration, considering inherited loggers.
* @return the name of the logger * @return the level configuration
* @since 2.7.13
*/ */
public String getName() { public LevelConfiguration getLevelConfiguration() {
return this.name; return getLevelConfiguration(ConfigurationScope.INHERITED);
}
/**
* Return the level configuration for the given scope.
* @param scope the configuration scope
* @return the level configuration or {@code null} for
* {@link ConfigurationScope#DIRECT direct scope} results without applied
* configuration
* @since 2.7.13
*/
public LevelConfiguration getLevelConfiguration(ConfigurationScope scope) {
return (scope != ConfigurationScope.DIRECT) ? this.inheritedLevelConfiguration : this.levelConfiguration;
} }
@Override @Override
@ -76,33 +119,131 @@ public final class LoggerConfiguration {
if (this == obj) { if (this == obj) {
return true; return true;
} }
if (obj == null) { if (obj == null || getClass() != obj.getClass()) {
return false; return false;
} }
if (obj instanceof LoggerConfiguration other) { LoggerConfiguration other = (LoggerConfiguration) obj;
boolean rtn = true; return ObjectUtils.nullSafeEquals(this.name, other.name)
rtn = rtn && ObjectUtils.nullSafeEquals(this.name, other.name); && ObjectUtils.nullSafeEquals(this.levelConfiguration, other.levelConfiguration)
rtn = rtn && ObjectUtils.nullSafeEquals(this.configuredLevel, other.configuredLevel); && ObjectUtils.nullSafeEquals(this.inheritedLevelConfiguration, other.inheritedLevelConfiguration);
rtn = rtn && ObjectUtils.nullSafeEquals(this.effectiveLevel, other.effectiveLevel);
return rtn;
}
return super.equals(obj);
} }
@Override @Override
public int hashCode() { public int hashCode() {
final int prime = 31; return Objects.hash(this.name, this.levelConfiguration, this.inheritedLevelConfiguration);
int result = 1;
result = prime * result + ObjectUtils.nullSafeHashCode(this.name);
result = prime * result + ObjectUtils.nullSafeHashCode(this.configuredLevel);
result = prime * result + ObjectUtils.nullSafeHashCode(this.effectiveLevel);
return result;
} }
@Override @Override
public String toString() { public String toString() {
return "LoggerConfiguration [name=" + this.name + ", configuredLevel=" + this.configuredLevel return "LoggerConfiguration [name=" + this.name + ", levelConfiguration=" + this.levelConfiguration
+ ", effectiveLevel=" + this.effectiveLevel + "]"; + ", inheritedLevelConfiguration=" + this.inheritedLevelConfiguration + "]";
}
/**
* Supported logger configurations scopes.
*
* @since 2.7.13
*/
public enum ConfigurationScope {
/**
* Only return configuration that has been applied directly applied. Often
* referred to as 'configured' or 'assigned' configuration.
*/
DIRECT,
/**
* May return configuration that has been applied to a parent logger. Often
* referred to as 'effective' configuration.
*/
INHERITED
}
/**
* Logger level configuration.
*
* @since 2.7.13
*/
public static final class LevelConfiguration {
private final String name;
private final LogLevel logLevel;
private LevelConfiguration(String name, LogLevel logLevel) {
this.name = name;
this.logLevel = logLevel;
}
/**
* Return the name of the level.
* @return the level name
*/
public String getName() {
return this.name;
}
/**
* Return the actual level value if possible.
* @return the level value
* @throws IllegalStateException if this is a {@link #isCustom() custom} level
*/
public LogLevel getLevel() {
Assert.state(this.logLevel != null, "Unable to provide LogLevel for '" + this.name + "'");
return this.logLevel;
}
/**
* Return if this is a custom level and cannot be represented by {@link LogLevel}.
* @return if this is a custom level
*/
public boolean isCustom() {
return this.logLevel == null;
}
@Override
public boolean equals(Object obj) {
if (this == obj) {
return true;
}
if (obj == null || getClass() != obj.getClass()) {
return false;
}
LevelConfiguration other = (LevelConfiguration) obj;
return this.logLevel == other.logLevel && ObjectUtils.nullSafeEquals(this.name, other.name);
}
@Override
public int hashCode() {
return Objects.hash(this.logLevel, this.name);
}
@Override
public String toString() {
return "LevelConfiguration [name=" + this.name + ", logLevel=" + this.logLevel + "]";
}
/**
* Create a new {@link LevelConfiguration} instance of the given {@link LogLevel}.
* @param logLevel the log level
* @return a new {@link LevelConfiguration} instance
*/
public static LevelConfiguration of(LogLevel logLevel) {
Assert.notNull(logLevel, "LogLevel must not be null");
return new LevelConfiguration(logLevel.name(), logLevel);
}
/**
* Create a new {@link LevelConfiguration} instance for a custom level name.
* @param name the log level name
* @return a new {@link LevelConfiguration} instance
*/
public static LevelConfiguration ofCustom(String name) {
Assert.hasText(name, "Name must not be empty");
return new LevelConfiguration(name, null);
}
} }
} }

@ -26,7 +26,8 @@ import java.util.concurrent.ConcurrentHashMap;
* *
* @author HaiTao Zhang * @author HaiTao Zhang
* @author Phillip Webb * @author Phillip Webb
* @since 2.2.0 #see {@link LoggerGroup} * @since 2.2.0
* @see LoggerGroup
*/ */
public final class LoggerGroups implements Iterable<LoggerGroup> { public final class LoggerGroups implements Iterable<LoggerGroup> {

@ -59,6 +59,7 @@ import org.springframework.boot.logging.AbstractLoggingSystem;
import org.springframework.boot.logging.LogFile; import org.springframework.boot.logging.LogFile;
import org.springframework.boot.logging.LogLevel; import org.springframework.boot.logging.LogLevel;
import org.springframework.boot.logging.LoggerConfiguration; import org.springframework.boot.logging.LoggerConfiguration;
import org.springframework.boot.logging.LoggerConfiguration.LevelConfiguration;
import org.springframework.boot.logging.LoggingInitializationContext; import org.springframework.boot.logging.LoggingInitializationContext;
import org.springframework.boot.logging.LoggingSystem; import org.springframework.boot.logging.LoggingSystem;
import org.springframework.boot.logging.LoggingSystemFactory; import org.springframework.boot.logging.LoggingSystemFactory;
@ -432,13 +433,18 @@ public class Log4J2LoggingSystem extends AbstractLoggingSystem {
if (loggerConfig == null) { if (loggerConfig == null) {
return null; return null;
} }
LogLevel level = LEVELS.convertNativeToSystem(loggerConfig.getLevel()); LevelConfiguration effectiveLevelConfiguration = getLevelConfiguration(loggerConfig.getLevel());
if (!StringUtils.hasLength(name) || LogManager.ROOT_LOGGER_NAME.equals(name)) { if (!StringUtils.hasLength(name) || LogManager.ROOT_LOGGER_NAME.equals(name)) {
name = ROOT_LOGGER_NAME; name = ROOT_LOGGER_NAME;
} }
boolean isLoggerConfigured = loggerConfig.getName().equals(name); boolean isAssigned = loggerConfig.getName().equals(name);
LogLevel configuredLevel = (isLoggerConfigured) ? level : null; LevelConfiguration assignedLevelConfiguration = (!isAssigned) ? null : effectiveLevelConfiguration;
return new LoggerConfiguration(name, configuredLevel, level); return new LoggerConfiguration(name, assignedLevelConfiguration, effectiveLevelConfiguration);
}
private LevelConfiguration getLevelConfiguration(Level level) {
LogLevel logLevel = LEVELS.convertNativeToSystem(level);
return (logLevel != null) ? LevelConfiguration.of(logLevel) : LevelConfiguration.ofCustom(level.name());
} }
@Override @Override

@ -0,0 +1,187 @@
/*
* Copyright 2012-2023 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.logging;
import org.junit.jupiter.api.Nested;
import org.junit.jupiter.api.Test;
import org.springframework.boot.logging.LoggerConfiguration.ConfigurationScope;
import org.springframework.boot.logging.LoggerConfiguration.LevelConfiguration;
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
import static org.assertj.core.api.Assertions.assertThatIllegalStateException;
/**
* Tests for {@link LoggerConfiguration}.
*
* @author Phillip Webb
*/
class LoggerConfigurationTests {
@Test
void createWithLogLevelWhenNameIsNullThrowsException() {
assertThatIllegalArgumentException().isThrownBy(() -> new LoggerConfiguration(null, null, LogLevel.DEBUG))
.withMessage("Name must not be null");
}
@Test
void createWithLogLevelWhenEffectiveLevelIsNullThrowsException() {
assertThatIllegalArgumentException().isThrownBy(() -> new LoggerConfiguration("test", null, (LogLevel) null))
.withMessage("EffectiveLevel must not be null");
}
@Test
void createWithLevelConfigurationWhenNameIsNullThrowsException() {
assertThatIllegalArgumentException()
.isThrownBy(() -> new LoggerConfiguration(null, null, LevelConfiguration.of(LogLevel.DEBUG)))
.withMessage("Name must not be null");
}
@Test
void createWithLevelConfigurationWhenEffectiveLevelIsNullThrowsException() {
assertThatIllegalArgumentException()
.isThrownBy(() -> new LoggerConfiguration("test", null, (LevelConfiguration) null))
.withMessage("EffectiveLevelConfiguration must not be null");
}
@Test
void getNameReturnsName() {
LoggerConfiguration configuration = new LoggerConfiguration("test", null,
LevelConfiguration.of(LogLevel.DEBUG));
assertThat(configuration.getName()).isEqualTo("test");
}
@Test
void getConfiguredLevelWhenConfiguredReturnsLevel() {
LoggerConfiguration configuration = new LoggerConfiguration("test", LevelConfiguration.of(LogLevel.DEBUG),
LevelConfiguration.of(LogLevel.DEBUG));
assertThat(configuration.getConfiguredLevel()).isEqualTo(LogLevel.DEBUG);
}
@Test
void getConfiguredLevelWhenNotConfiguredReturnsNull() {
LoggerConfiguration configuration = new LoggerConfiguration("test", null,
LevelConfiguration.of(LogLevel.DEBUG));
assertThat(configuration.getConfiguredLevel()).isNull();
}
@Test
void getEffectiveLevelReturnsEffectiveLevel() {
LoggerConfiguration configuration = new LoggerConfiguration("test", null,
LevelConfiguration.of(LogLevel.DEBUG));
assertThat(configuration.getEffectiveLevel()).isEqualTo(LogLevel.DEBUG);
}
@Test
void getLevelConfigurationWithDirectScopeWhenConfiguredReturnsConfiguration() {
LevelConfiguration assigned = LevelConfiguration.of(LogLevel.DEBUG);
LoggerConfiguration configuration = new LoggerConfiguration("test", assigned,
LevelConfiguration.of(LogLevel.DEBUG));
assertThat(configuration.getLevelConfiguration(ConfigurationScope.DIRECT)).isEqualTo(assigned);
}
@Test
void getLevelConfigurationWithDirectScopeWhenNotConfiguredReturnsNull() {
LoggerConfiguration configuration = new LoggerConfiguration("test", null,
LevelConfiguration.of(LogLevel.DEBUG));
assertThat(configuration.getLevelConfiguration(ConfigurationScope.DIRECT)).isNull();
}
@Test
void getLevelConfigurationWithInheritedScopeReturnsConfiguration() {
LevelConfiguration effective = LevelConfiguration.of(LogLevel.DEBUG);
LoggerConfiguration configuration = new LoggerConfiguration("test", null, effective);
assertThat(configuration.getLevelConfiguration(ConfigurationScope.INHERITED)).isEqualTo(effective);
}
/**
* Tests for {@link LevelConfiguration}.
*/
@Nested
class LevelConfigurationTests {
@Test
void ofWhenLogLevelIsNullThrowsException() {
assertThatIllegalArgumentException().isThrownBy(() -> LevelConfiguration.of(null))
.withMessage("LogLevel must not be null");
}
@Test
void ofCreatesConfiguration() {
LevelConfiguration configuration = LevelConfiguration.of(LogLevel.DEBUG);
assertThat(configuration.getLevel()).isEqualTo(LogLevel.DEBUG);
}
@Test
void ofCustomWhenNameIsNullThrowsException() {
assertThatIllegalArgumentException().isThrownBy(() -> LevelConfiguration.ofCustom(null))
.withMessage("Name must not be empty");
}
@Test
void ofCustomWhenNameIsEmptyThrowsException() {
assertThatIllegalArgumentException().isThrownBy(() -> LevelConfiguration.ofCustom(""))
.withMessage("Name must not be empty");
}
@Test
void ofCustomCreatesConfiguration() {
LevelConfiguration configuration = LevelConfiguration.ofCustom("FINE");
assertThat(configuration).isNotNull();
}
@Test
void getNameWhenFromLogLevelReturnsName() {
LevelConfiguration configuration = LevelConfiguration.of(LogLevel.DEBUG);
assertThat(configuration.getName()).isEqualTo("DEBUG");
}
@Test
void getNameWhenCustomReturnsName() {
LevelConfiguration configuration = LevelConfiguration.ofCustom("FINE");
assertThat(configuration.getName()).isEqualTo("FINE");
}
@Test
void getLevelWhenCustomThrowsException() {
LevelConfiguration configuration = LevelConfiguration.ofCustom("FINE");
assertThatIllegalStateException().isThrownBy(() -> configuration.getLevel())
.withMessage("Unable to provide LogLevel for 'FINE'");
}
@Test
void getLevelReturnsLevel() {
LevelConfiguration configuration = LevelConfiguration.of(LogLevel.DEBUG);
assertThat(configuration.getLevel()).isEqualTo(LogLevel.DEBUG);
}
@Test
void isCustomWhenNotCustomReturnsFalse() {
LevelConfiguration configuration = LevelConfiguration.of(LogLevel.DEBUG);
assertThat(configuration.isCustom()).isFalse();
}
@Test
void isCustomWhenCustomReturnsTrue() {
LevelConfiguration configuration = LevelConfiguration.ofCustom("DEBUG");
assertThat(configuration.isCustom()).isTrue();
}
}
}

@ -224,6 +224,18 @@ class Log4J2LoggingSystemTests extends AbstractLoggingSystemTests {
assertIsPresent("org.springframework.boot.logging.log4j2.Log4J2LoggingSystemTests$Nested", loggers, null); assertIsPresent("org.springframework.boot.logging.log4j2.Log4J2LoggingSystemTests$Nested", loggers, null);
} }
@Test // gh-35227
void getLoggingConfigurationsWhenHasCustomLevel() {
this.loggingSystem.beforeInitialize();
this.loggingSystem.initialize(this.initializationContext, null, null);
LoggerContext loggerContext = (LoggerContext) LogManager.getContext(false);
String loggerName = getClass().getName();
org.apache.logging.log4j.Level level = org.apache.logging.log4j.Level.forName("CUSTOM_LEVEL", 1000);
loggerContext.getConfiguration().addLogger(loggerName, new LoggerConfig(loggerName, level, true));
LoggerConfiguration configuration = this.loggingSystem.getLoggerConfiguration(loggerName);
assertThat(configuration.getLevelConfiguration().getName()).isEqualTo("CUSTOM_LEVEL");
}
private void assertIsPresent(String loggerName, Map<String, LogLevel> loggers, LogLevel logLevel) { private void assertIsPresent(String loggerName, Map<String, LogLevel> loggers, LogLevel logLevel) {
assertThat(loggers).containsKey(loggerName); assertThat(loggers).containsKey(loggerName);
assertThat(loggers).containsEntry(loggerName, logLevel); assertThat(loggers).containsEntry(loggerName, logLevel);

Loading…
Cancel
Save