Skip to content

S
spring-boot
Commit a85e27c7 authored by Stephane Nicoll's avatar Stephane Nicoll
Browse files
Options

Further replace magic number by HttpStatus 

See gh-15130
parent 9ee63038
Show whitespace changes
Inline Side-by-side
Showing with 14 additions and 12 deletions
spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/metrics/web/servlet/WebMvcTags.java
View file @ a85e27c7
...@@ -180,19 +180,21 @@ public final class WebMvcTags { ...@@ -180,19 +180,21 @@ public final class WebMvcTags {
*/ */
public static Tag outcome(HttpServletResponse response) { public static Tag outcome(HttpServletResponse response) {
if (response != null) { if (response != null) {
int status = response.getStatus(); HttpStatus status = extractStatus(response);
if (status < 200) { if (status != null) {
if (status.is1xxInformational()) {
return OUTCOME_INFORMATIONAL; return OUTCOME_INFORMATIONAL;
} }
if (status < 300) { if (status.is2xxSuccessful()) {
return OUTCOME_SUCCESS; return OUTCOME_SUCCESS;
} }
if (status < 400) { if (status.is3xxRedirection()) {
return OUTCOME_REDIRECTION; return OUTCOME_REDIRECTION;
} }
if (status < 500) { if (status.is4xxClientError()) {
return OUTCOME_CLIENT_ERROR; return OUTCOME_CLIENT_ERROR;
} }
}
return OUTCOME_SERVER_ERROR; return OUTCOME_SERVER_ERROR;
} }
return OUTCOME_UNKNOWN; return OUTCOME_UNKNOWN;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment