mmoayyed opened a new pull request #301: URL: https://github.com/apache/syncope/pull/301
This pull request starts to remove CGLib proxies from Configuration classes. The changes are quite large, but they for the most part tend to be cosmetic. Here is a summary: 1. `proxyBeanMethods` is now set to false for `@Configuration` annotations 2. As a result, bean methods can no longer call each other directly, since there is no proxy to intercept. Therefor, intra-bean calls have been refactored to use parameter injection. 3. Where possible, `@Component` annotations are removed and the component creation has been replaced with a `@Bean` instead. Furthermore, field injections that contain auto-wired references are also removed from such components, and are replaced with c-tor injection at creation time. 4. `@Autowired` annotations are removed from `@Bean` methods; `@Resource` annotations are also replaced with `@Qualifier`. 5. In order to assist with PRs running CI builds on forks, CI builds can now also trigger for PRs as long as the PR branch name begins with `pr-` 6. CAS build is now switched to use RC4. This is only the first part of this effort; CGLib proxies are still created for many `@Bean`s and `@Component` references, in scenarios where the bean does not have a parent interface, or is created with a concrete type. This would need to change, which also impacts how `@ConditionalOnClass` elements are used. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
