Browse Source

Merge branch 'gh-3628' into 1.2.x

pull/3661/head
Andy Wilkinson 11 years ago
parent
commit
acfb07bdd6
  1. 9
      spring-boot/src/main/java/org/springframework/boot/logging/LoggingApplicationListener.java
  2. 12
      spring-boot/src/test/java/org/springframework/boot/logging/LoggingApplicationListenerTests.java

9
spring-boot/src/main/java/org/springframework/boot/logging/LoggingApplicationListener.java

@ -246,13 +246,20 @@ public class LoggingApplicationListener implements SmartApplicationListener { @@ -246,13 +246,20 @@ public class LoggingApplicationListener implements SmartApplicationListener {
name = null;
}
level = environment.resolvePlaceholders(level);
system.setLogLevel(name, LogLevel.valueOf(level.toUpperCase()));
system.setLogLevel(name, coerceLogLevel(level));
}
catch (RuntimeException ex) {
this.logger.error("Cannot set level: " + level + " for '" + name + "'");
}
}
private LogLevel coerceLogLevel(String level) {
if ("false".equalsIgnoreCase(level)) {
return LogLevel.OFF;
}
return LogLevel.valueOf(level.toUpperCase());
}
public void setOrder(int order) {
this.order = order;
}

12
spring-boot/src/test/java/org/springframework/boot/logging/LoggingApplicationListenerTests.java

@ -258,6 +258,18 @@ public class LoggingApplicationListenerTests { @@ -258,6 +258,18 @@ public class LoggingApplicationListenerTests {
assertThat(this.outputCapture.toString(), not(containsString("testatfatal")));
}
@Test
public void parseLevelsMapsFalseToOff() throws Exception {
EnvironmentTestUtils.addEnvironment(this.context,
"logging.level.org.springframework.boot=false");
this.initializer.initialize(this.context.getEnvironment(),
this.context.getClassLoader());
this.logger.debug("testatdebug");
this.logger.fatal("testatfatal");
assertThat(this.outputCapture.toString(), not(containsString("testatdebug")));
assertThat(this.outputCapture.toString(), not(containsString("testatfatal")));
}
@Test
public void parseArgsDisabled() throws Exception {
this.initializer.setParseArgs(false);

Loading…
Cancel
Save