mmoayyed commented on a change in pull request #301:
URL: https://github.com/apache/syncope/pull/301#discussion_r777350356



##########
File path: 
core/starter/src/main/java/org/apache/syncope/core/starter/SyncopeCoreApplication.java
##########
@@ -105,4 +221,31 @@ public SyncopeCoreStart keymasterStart(final DomainHolder 
domainHolder) {
     public KeymasterStop keymasterStop() {
         return new KeymasterStop(NetworkService.Type.CORE);
     }
+

Review comment:
       Yes, I definitely agree. That would be a fairly comprehensive change, 
likely to impact many modules and I plan to tackle that separately via a series 
of PRs. That makes it easier to test incrementally as well, and the only *sure* 
way I know to test is to look at the beans actuator endpoint and make sure no 
reference to "CGLIB" is found in the output. It's fairly slow, but certain :) 
   




-- 
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]


Reply via email to