Change query in MongoHealthIndicator

This commit changes the query in MongoHealthIndicator from serverStatus to buildInfo to avoid unprivileged access and corresponding errors.

fixes #1289

Author: cdupuis

spring-boot-actuator/src/main/java/org/springframework/boot/actuate/health/MongoHealthIndicator.java

@@ -24,7 +24,7 @@
 /**
  * Simple implementation of a {@link HealthIndicator} returning status information for
  * Mongo data stores.
- *
+ * 
  * @author Christian Dupuis
  * @since 1.1.0
  */
@@ -39,7 +39,7 @@ public MongoHealthIndicator(MongoTemplate mongoTemplate) {
 
 	@Override
 	protected void doHealthCheck(Health.Builder builder) throws Exception {
-		CommandResult result = this.mongoTemplate.executeCommand("{ serverStatus: 1 }");
+		CommandResult result = this.mongoTemplate.executeCommand("{ buildInfo: 1 }");
 		builder.up().withDetail("version", result.getString("version"));
 	}
 

spring-boot-actuator/src/test/java/org/springframework/boot/actuate/health/MongoHealthIndicatorTests.java

@@ -36,7 +36,7 @@
 
 /**
  * Tests for {@link MongoHealthIndicator}.
- *
+ * 
  * @author Christian Dupuis
  */
 public class MongoHealthIndicatorTests {
@@ -67,7 +67,7 @@ public void mongoIsUp() throws Exception {
 		CommandResult commandResult = Mockito.mock(CommandResult.class);
 		Mockito.when(commandResult.getString("version")).thenReturn("2.6.4");
 		MongoTemplate mongoTemplate = Mockito.mock(MongoTemplate.class);
-		Mockito.when(mongoTemplate.executeCommand("{ serverStatus: 1 }")).thenReturn(
+		Mockito.when(mongoTemplate.executeCommand("{ buildInfo: 1 }")).thenReturn(
 				commandResult);
 		MongoHealthIndicator healthIndicator = new MongoHealthIndicator(mongoTemplate);
 
@@ -76,13 +76,13 @@ public void mongoIsUp() throws Exception {
 		assertEquals("2.6.4", health.getDetails().get("version"));
 
 		Mockito.verify(commandResult).getString("version");
-		Mockito.verify(mongoTemplate).executeCommand("{ serverStatus: 1 }");
+		Mockito.verify(mongoTemplate).executeCommand("{ buildInfo: 1 }");
 	}
 
 	@Test
 	public void mongoIsDown() throws Exception {
 		MongoTemplate mongoTemplate = Mockito.mock(MongoTemplate.class);
-		Mockito.when(mongoTemplate.executeCommand("{ serverStatus: 1 }")).thenThrow(
+		Mockito.when(mongoTemplate.executeCommand("{ buildInfo: 1 }")).thenThrow(
 				new MongoException("Connection failed"));
 		MongoHealthIndicator healthIndicator = new MongoHealthIndicator(mongoTemplate);
 
@@ -91,6 +91,6 @@ public void mongoIsDown() throws Exception {
 		assertTrue(((String) health.getDetails().get("error"))
 				.contains("Connection failed"));
 
-		Mockito.verify(mongoTemplate).executeCommand("{ serverStatus: 1 }");
+		Mockito.verify(mongoTemplate).executeCommand("{ buildInfo: 1 }");
 	}
 }