Improve health contributor null support

Update `NamedContributorsMapAdapter` to check for `null` keys or values
during construction. Also update `HealthEndpointSupport` to allow
null component entries.

See gh-18687

Author: philwebb

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

@@ -134,15 +134,19 @@ protected abstract T aggregateContributions(ApiVersion apiVersion, Map<String, T
 
 	protected final CompositeHealth getCompositeHealth(ApiVersion apiVersion, Map<String, HealthComponent> components,
 			StatusAggregator statusAggregator, boolean showComponents, Set<String> groupNames) {
-		Status status = statusAggregator.getAggregateStatus(
-				components.values().stream().map(HealthComponent::getStatus).collect(Collectors.toSet()));
+		Status status = statusAggregator
+				.getAggregateStatus(components.values().stream().map(this::getStatus).collect(Collectors.toSet()));
 		Map<String, HealthComponent> instances = showComponents ? components : null;
 		if (groupNames != null) {
 			return new SystemHealth(apiVersion, status, instances, groupNames);
 		}
 		return new CompositeHealth(apiVersion, status, instances);
 	}
 
+	private Status getStatus(HealthComponent component) {
+		return (component != null) ? component.getStatus() : Status.UNKNOWN;
+	}
+
 	/**
 	 * A health result containing health and the group that created it.
 	 *

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

@@ -43,6 +43,9 @@ abstract class NamedContributorsMapAdapter<V, C> implements NamedContributors<C>
 	NamedContributorsMapAdapter(Map<String, V> map, Function<V, ? extends C> valueAdapter) {
 		Assert.notNull(map, "Map must not be null");
 		Assert.notNull(valueAdapter, "ValueAdapter must not be null");
+		map.keySet().stream().forEach((key) -> Assert.notNull(key, "Map must not contain null keys"));
+		map.values().stream().map(valueAdapter)
+				.forEach((value) -> Assert.notNull(value, "Map must not contain null values"));
 		this.map = Collections.unmodifiableMap(new LinkedHashMap<>(map));
 		this.valueAdapter = valueAdapter;
 	}

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

@@ -48,6 +48,22 @@ void createWhenValueAdapterIsNullThrowsException() {
 				.withMessage("ValueAdapter must not be null");
 	}
 
+	@Test
+	void createWhenMapContainsNullValueThrowsException() {
+		assertThatIllegalArgumentException()
+				.isThrownBy(() -> new TestNamedContributorsMapAdapter<>(Collections.singletonMap("test", null),
+						Function.identity()))
+				.withMessage("Map must not contain null values");
+	}
+
+	@Test
+	void createWhenMapContainsNullKeyThrowsException() {
+		assertThatIllegalArgumentException()
+				.isThrownBy(() -> new TestNamedContributorsMapAdapter<>(Collections.singletonMap(null, "test"),
+						Function.identity()))
+				.withMessage("Map must not contain null keys");
+	}
+
 	@Test
 	void iterateReturnsAdaptedEntries() {
 		TestNamedContributorsMapAdapter<String> adapter = createAdapter();