Commit f9aeb6ae authored by Christian Dupuis's avatar Christian Dupuis

Polish

parent 344fb461
...@@ -109,8 +109,8 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware, ...@@ -109,8 +109,8 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware,
@Bean @Bean
@ConditionalOnMissingBean @ConditionalOnMissingBean
public EndpointHandlerMapping endpointHandlerMapping() { public EndpointHandlerMapping endpointHandlerMapping() {
EndpointHandlerMapping mapping = new EndpointHandlerMapping( EndpointHandlerMapping mapping = new EndpointHandlerMapping(mvcEndpoints()
mvcEndpoints().getEndpoints()); .getEndpoints());
boolean disabled = ManagementServerPort.get(this.applicationContext) != ManagementServerPort.SAME; boolean disabled = ManagementServerPort.get(this.applicationContext) != ManagementServerPort.SAME;
mapping.setDisabled(disabled); mapping.setDisabled(disabled);
if (!disabled) { if (!disabled) {
...@@ -169,7 +169,8 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware, ...@@ -169,7 +169,8 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware,
public HealthMvcEndpoint healthMvcEndpoint(HealthEndpoint delegate) { public HealthMvcEndpoint healthMvcEndpoint(HealthEndpoint delegate) {
HealthMvcEndpoint healthMvcEndpoint = new HealthMvcEndpoint(delegate); HealthMvcEndpoint healthMvcEndpoint = new HealthMvcEndpoint(delegate);
if (this.healthMvcEndpointProperties.getMapping() != null) { if (this.healthMvcEndpointProperties.getMapping() != null) {
healthMvcEndpoint.setStatusMapping(this.healthMvcEndpointProperties.getMapping()); healthMvcEndpoint.setStatusMapping(this.healthMvcEndpointProperties
.getMapping());
} }
return healthMvcEndpoint; return healthMvcEndpoint;
} }
...@@ -205,7 +206,8 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware, ...@@ -205,7 +206,8 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware,
// Ensure close on the parent also closes the child // Ensure close on the parent also closes the child
if (this.applicationContext instanceof ConfigurableApplicationContext) { if (this.applicationContext instanceof ConfigurableApplicationContext) {
((ConfigurableApplicationContext) this.applicationContext).addApplicationListener(new ApplicationListener<ContextClosedEvent>() { ((ConfigurableApplicationContext) this.applicationContext)
.addApplicationListener(new ApplicationListener<ContextClosedEvent>() {
@Override @Override
public void onApplicationEvent(ContextClosedEvent event) { public void onApplicationEvent(ContextClosedEvent event) {
...@@ -217,13 +219,15 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware, ...@@ -217,13 +219,15 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware,
} }
try { try {
childContext.refresh(); childContext.refresh();
} catch (RuntimeException ex) { }
catch (RuntimeException ex) {
// No support currently for deploying a war with management.port=<different>, // No support currently for deploying a war with management.port=<different>,
// and this is the signature of that happening // and this is the signature of that happening
if (ex instanceof EmbeddedServletContainerException if (ex instanceof EmbeddedServletContainerException
|| ex.getCause() instanceof EmbeddedServletContainerException) { || ex.getCause() instanceof EmbeddedServletContainerException) {
logger.warn("Could not start embedded container (management endpoints are still available through JMX)"); logger.warn("Could not start embedded container (management endpoints are still available through JMX)");
} else { }
else {
throw ex; throw ex;
} }
} }
...@@ -238,14 +242,17 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware, ...@@ -238,14 +242,17 @@ public class EndpointWebMvcAutoConfiguration implements ApplicationContextAware,
ServerProperties serverProperties; ServerProperties serverProperties;
try { try {
serverProperties = beanFactory.getBean(ServerProperties.class); serverProperties = beanFactory.getBean(ServerProperties.class);
} catch (NoSuchBeanDefinitionException ex) { }
catch (NoSuchBeanDefinitionException ex) {
serverProperties = new ServerProperties(); serverProperties = new ServerProperties();
} }
ManagementServerProperties managementServerProperties; ManagementServerProperties managementServerProperties;
try { try {
managementServerProperties = beanFactory.getBean(ManagementServerProperties.class); managementServerProperties = beanFactory
} catch (NoSuchBeanDefinitionException ex) { .getBean(ManagementServerProperties.class);
}
catch (NoSuchBeanDefinitionException ex) {
managementServerProperties = new ManagementServerProperties(); managementServerProperties = new ManagementServerProperties();
} }
......
...@@ -48,7 +48,8 @@ public class HealthEndpoint extends AbstractEndpoint<Health> { ...@@ -48,7 +48,8 @@ public class HealthEndpoint extends AbstractEndpoint<Health> {
if (healthIndicators.size() == 1) { if (healthIndicators.size() == 1) {
this.healthIndicator = healthIndicators.values().iterator().next(); this.healthIndicator = healthIndicators.values().iterator().next();
} else { }
else {
CompositeHealthIndicator healthIndicator = new CompositeHealthIndicator( CompositeHealthIndicator healthIndicator = new CompositeHealthIndicator(
healthAggregator); healthAggregator);
for (Map.Entry<String, HealthIndicator> h : healthIndicators.entrySet()) { for (Map.Entry<String, HealthIndicator> h : healthIndicators.entrySet()) {
......
...@@ -67,8 +67,10 @@ public class OrderedHealthAggregator implements HealthAggregator { ...@@ -67,8 +67,10 @@ public class OrderedHealthAggregator implements HealthAggregator {
@Override @Override
public int compare(Status s1, Status s2) { public int compare(Status s1, Status s2) {
return Integer.valueOf( return Integer.valueOf(
OrderedHealthAggregator.this.statusOrder.indexOf(s1.getCode())).compareTo( OrderedHealthAggregator.this.statusOrder.indexOf(s1.getCode()))
Integer.valueOf(OrderedHealthAggregator.this.statusOrder.indexOf(s2.getCode()))); .compareTo(
Integer.valueOf(OrderedHealthAggregator.this.statusOrder
.indexOf(s2.getCode())));
} }
}); });
......
...@@ -88,7 +88,8 @@ public class SimpleDataSourceHealthIndicator implements HealthIndicator { ...@@ -88,7 +88,8 @@ public class SimpleDataSourceHealthIndicator implements HealthIndicator {
} }
}); });
health.withDetail("database", product); health.withDetail("database", product);
} catch (DataAccessException ex) { }
catch (DataAccessException ex) {
health.down().withException(ex); health.down().withException(ex);
} }
String query = detectQuery(product); String query = detectQuery(product);
...@@ -96,7 +97,8 @@ public class SimpleDataSourceHealthIndicator implements HealthIndicator { ...@@ -96,7 +97,8 @@ public class SimpleDataSourceHealthIndicator implements HealthIndicator {
try { try {
health.withDetail("hello", health.withDetail("hello",
this.jdbcTemplate.queryForObject(query, Object.class)); this.jdbcTemplate.queryForObject(query, Object.class));
} catch (Exception ex) { }
catch (Exception ex) {
health.down().withException(ex); health.down().withException(ex);
} }
} }
......
...@@ -16,7 +16,6 @@ ...@@ -16,7 +16,6 @@
package org.springframework.boot.actuate.health; package org.springframework.boot.actuate.health;
/** /**
* Default implementation of {@link HealthIndicator} that simply returns {@literal "ok"}. * Default implementation of {@link HealthIndicator} that simply returns {@literal "ok"}.
* *
......
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