Skip to content

Add ignored paths for config monitor - #3296

Merged
ryanjbaxter merged 1 commit into
spring-cloud:5.0.xfrom
yashdotdev13:fix/1885-ignore-unrelated-refresh-paths-5.0.x
Sep 24, 2026
Merged

ryanjbaxter merged 1 commit into
spring-cloud:5.0.xfrom
yashdotdev13:fix/1885-ignore-unrelated-refresh-paths-5.0.x

Conversation

@yashdotdev13

Copy link
Copy Markdown
Contributor

Summary

  • Add spring.cloud.config.server.monitor.ignored-paths for excluding repository paths from refresh notifications.
  • Filter ignored paths before resolving service names.
  • Add test coverage for ignored and non-ignored paths.
  • Document the new monitor property.

This is the 5.0.x compatible version of the change from #1885.

this.http = http;
}

public static class Http {

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.

What is this for?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Http is pre-existing configuration and isn't part of this change. My change only adds the ignoredPaths property and its getter/setter to MonitorConfigurationProperties.

@yashdotdev13
yashdotdev13 force-pushed the fix/1885-ignore-unrelated-refresh-paths-5.0.x branch from bb8ee01 to 575b774 Compare September 18, 2026 17:23
@ryanjbaxter

Copy link
Copy Markdown
Contributor

I think you merged main into your branch on accident

Signed-off-by: 98001yash <yashchauhan.gaya@gmail.com>
@yashdotdev13
yashdotdev13 force-pushed the fix/1885-ignore-unrelated-refresh-paths-5.0.x branch from 575b774 to 4ae9568 Compare September 18, 2026 18:27
@yashdotdev13

Copy link
Copy Markdown
Contributor Author

Thanks Ryan. I did have some unrelated history on the branch previously. I cleaned up the branch and force-updated the existing PR so it now contains only the #1885 change on top of 5.0.x. The current PR has a single commit (4ae9568) with the six relevant files.

@yashdotdev13

Copy link
Copy Markdown
Contributor Author

The branch has now been cleaned up and the updated commit is passing both DCO and CI checks. Please take another look when you get a chance.

@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.

Looks good but we will have to wait to merge this until after the 24th when we do our release

@yashdotdev13

Copy link
Copy Markdown
Contributor Author

Thanks, sounds good. I’ll wait until after the release.

@ryanjbaxter
ryanjbaxter merged commit 01349ec into spring-cloud:5.0.x Sep 24, 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.

3 participants