Skip to content

Context refresh fails on reactive Cloud Foundry when using Actuator without spring-boot-health#50916

Open
dlwldnjs1009 wants to merge 1 commit into
spring-projects:4.0.xfrom
dlwldnjs1009:cf-reactive-health-absent-4.0.x
Open

Context refresh fails on reactive Cloud Foundry when using Actuator without spring-boot-health#50916
dlwldnjs1009 wants to merge 1 commit into
spring-projects:4.0.xfrom
dlwldnjs1009:cf-reactive-health-absent-4.0.x

Conversation

@dlwldnjs1009

Copy link
Copy Markdown
Contributor

Follow-up to #50857, which fixed the servlet Cloud Foundry actuator
auto-configuration. #50794 noted
CloudFoundryReactiveActuatorAutoConfiguration as an unverified sibling.

This PR confirms the reactive sibling with a failing test. When
spring-boot-health is absent, parsing the reactive Cloud Foundry actuator
configuration resolves ReactiveHealthEndpointWebExtension from the health
@Bean method signature, causing context refresh to fail before
/cloudfoundryapplication can be registered.

Changes

  • Move cloudFoundryReactiveHealthEndpointWebExtension into a nested
    HealthConfiguration guarded by @ConditionalOnClass(HealthEndpoint.class)

Testing

  • CloudFoundryReactiveActuatorAutoConfigurationTests.refreshSucceedsWithoutHealth (new; fails before this change)
  • CloudFoundryReactiveActuatorAutoConfigurationTests and CloudFoundryActuatorAutoConfigurationTests pass

Move the reactive Cloud Foundry health endpoint extension into a
health-specific configuration guarded by @ConditionalOnClass. When
spring-boot-health is absent, scanning the outer configuration attempts
to resolve ReactiveHealthEndpointWebExtension and causes context refresh
to fail.

Signed-off-by: Lee JiWon <dlwldnjs1009@gmail.com>
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: waiting-for-triage An issue we've not yet triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants