From 82ac8b5ca13be1de2ebafe0b0d76d8c96aed050e Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Tue, 11 Apr 2017 13:29:53 +0200 Subject: [PATCH] Polish "Fix http status code with unknown log level" Closes gh-8813 --- .../boot/actuate/endpoint/mvc/LoggersMvcEndpoint.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/LoggersMvcEndpoint.java b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/LoggersMvcEndpoint.java index d582330442..c774ce1ab9 100644 --- a/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/LoggersMvcEndpoint.java +++ b/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/mvc/LoggersMvcEndpoint.java @@ -68,15 +68,17 @@ public class LoggersMvcEndpoint extends EndpointMvcAdapter { // disabled return getDisabledResponse(); } - String level = configuration.get("configuredLevel"); + LogLevel logLevel; try { - LogLevel logLevel = level == null ? null : LogLevel.valueOf(level.toUpperCase()); - this.delegate.setLogLevel(name, logLevel); - return ResponseEntity.ok().build(); + String level = configuration.get("configuredLevel"); + logLevel = level == null ? null : LogLevel.valueOf(level.toUpperCase()); } catch (IllegalArgumentException ex) { return ResponseEntity.badRequest().build(); } + + this.delegate.setLogLevel(name, logLevel); + return ResponseEntity.ok().build(); } }