Skip to content

patch: solve issue with beanpostprocessor not eligable #98

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 18, 2024

Conversation

rspiegl
Copy link
Contributor

@rspiegl rspiegl commented Apr 17, 2024

In the caching module we refactored the conditionals to limit the auto configuration. Due to this the implementation of the CachingConfigurer (to get a custom CacheErrorHandler) needed a dependency on RedisConnectionFactory or else it wouldn't be able to be post processed (due to the CachingConfigurer being needed in a MeterRegistryAutoConfiguration).
This PR limits the dependency on RedisConnectionFactory to the CachingConfigurer and thus makes it easier for Spring to instantiate the beans.

@rspiegl rspiegl merged commit 60c2f7a into master Apr 18, 2024
2 checks passed
@rspiegl rspiegl deleted the rs/fix-caching-module branch April 18, 2024 08:54
@rspiegl rspiegl changed the title patch: limit dependency of conditionals to smallest scope in caching patch: solve issue with beanpostprocessor not eligable Apr 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant