Add New Rule to Ensure Configuration Classes Are Records #220
Conversation
|
|
Hi @mnhock - Why do you think configuration classes should be records? I couldn't find anything regarding the real why in the documentation. I am asking because records are final classes and cant be extended by cglib as a cglib proxy. Therefore, one must really pay attention when using bean method-handles to reference spring beans. Of course, I am aware that all @AutoConfigurations do implement proxyBeanMethods=false but this was a deliberate choice of the Spring Boot team, to not use bean method references. For performance reasons. Maybe checking for proxyBeanMethods=false and not using the bean methods at all, would be a good addition. |
|
Hi @patbaumgartner! Thank you so much to point this out! This was planned for Properties not for Configuration classes. I will change this. THANKS |
|
Yay! Thanks @mnhock - I think it fits for @ConfigurationProperties without any limitation issues. :)) |
Closes #219