Merge pull request #12419 from izeye:map-status
* pr/12419: Invoke mapStatus() only when necessary
This commit is contained in:
commit
7278d478bf
|
|
@ -65,13 +65,13 @@ public class HealthWebEndpointResponseMapper {
|
||||||
*/
|
*/
|
||||||
public WebEndpointResponse<Health> map(Health health, SecurityContext securityContext,
|
public WebEndpointResponse<Health> map(Health health, SecurityContext securityContext,
|
||||||
ShowDetails showDetails) {
|
ShowDetails showDetails) {
|
||||||
Integer status = this.statusHttpMapper.mapStatus(health.getStatus());
|
|
||||||
if (showDetails == ShowDetails.NEVER
|
if (showDetails == ShowDetails.NEVER
|
||||||
|| (showDetails == ShowDetails.WHEN_AUTHORIZED
|
|| (showDetails == ShowDetails.WHEN_AUTHORIZED
|
||||||
&& (securityContext.getPrincipal() == null
|
&& (securityContext.getPrincipal() == null
|
||||||
|| !isUserInRole(securityContext)))) {
|
|| !isUserInRole(securityContext)))) {
|
||||||
health = Health.status(health.getStatus()).build();
|
health = Health.status(health.getStatus()).build();
|
||||||
}
|
}
|
||||||
|
Integer status = this.statusHttpMapper.mapStatus(health.getStatus());
|
||||||
return new WebEndpointResponse<>(health, status);
|
return new WebEndpointResponse<>(health, status);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue