Skip to content

Add HTTP property path notifier - #3279

Merged
ryanjbaxter merged 5 commits into
spring-cloud:mainfrom
yashdotdev13:issue-2152-http-client-events
Sep 3, 2026
Merged

Add HTTP property path notifier#3279
ryanjbaxter merged 5 commits into
spring-cloud:mainfrom
yashdotdev13:issue-2152-http-client-events

Conversation

@yashdotdev13

Copy link
Copy Markdown
Contributor

Closes #2152

Summary

Adds support for notifying applications through HTTP when a configuration change is detected.

The existing PropertyPathEndpoint now supports multiple PropertyPathNotifier implementations, allowing the existing Spring Cloud Bus notifier and the new HTTP notifier to coexist.

Changes

  • Added PropertyPathNotifier abstraction.
  • Added BusPropertyPathNotifier for Spring Cloud Bus notifications.
  • Added HttpPropertyPathNotifier for HTTP-based notifications.
  • Added conditional auto-configuration for the HTTP notifier via:
    spring.cloud.config.server.monitor.http.enabled=true.
  • Updated PropertyPathEndpoint to notify all configured notifiers.
  • Preserved the existing single-notifier constructors for compatibility.
  • Added tests for the Bus and HTTP notifier implementations.
  • Added auto-configuration coverage for the HTTP notifier.

Testing

  • Ran the complete spring-cloud-config-monitor test suite.
  • 117 tests passed with 0 failures and 0 errors.
  • Checkstyle passed with 0 violations.

Signed-off-by: 98001yash <yashchauhan.gaya@gmail.com>

@ryanjbaxter ryanjbaxter left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We will need documentation as well

Signed-off-by: 98001yash <yashchauhan.gaya@gmail.com>
Comment thread docs/modules/ROOT/pages/server/push-notifications-and-bus.adoc
@ryanjbaxter

ryanjbaxter commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Also I think the bus dependncy should be made optional

<dependency>
<groupId>org.springframework.cloud</groupId>
<artifactId>spring-cloud-bus</artifactId>
</dependency>

@ryanjbaxter
ryanjbaxter merged commit 7d0d4de into spring-cloud:main Sep 3, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support sending events to clients with HTTP instead of AMPQ / kafka with Spring Cloud Config Monitor

3 participants