You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@syncope.apache.org by GitBox <gi...@apache.org> on 2021/09/17 11:17:11 UTC
[GitHub] [syncope] mmoayyed opened a new pull request #290: NOJIRA - Turn off bean proxying for WA; fixes typo
mmoayyed opened a new pull request #290:
URL: https://github.com/apache/syncope/pull/290
This pull request turns off and disables CGLIB bean proxying for the WA module. This is the recommended practice from Spring which would help WA in the future to compile better and easier against GraalVM. Without bean proxying, bean methods can no longer call each other directly during construction, as there is no proxy to intercept. Instead, references either need to be passed as auto-wired parameters (the approach used here), or they need to be injected into configuration classes using injections. (The latter approach is generally more preferable, if the number of injected parameters grows unwieldy)
Also fixes a bean name typo (missing `e`).
This approach might also be used for #287 as well.
--
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: dev-unsubscribe@syncope.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [syncope] mmoayyed merged pull request #290: NOJIRA - Turn off bean proxying for WA; fixes typo
Posted by GitBox <gi...@apache.org>.
mmoayyed merged pull request #290:
URL: https://github.com/apache/syncope/pull/290
--
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: dev-unsubscribe@syncope.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [syncope] ilgrosso commented on pull request #290: NOJIRA - Turn off bean proxying for WA; fixes typo
Posted by GitBox <gi...@apache.org>.
ilgrosso commented on pull request #290:
URL: https://github.com/apache/syncope/pull/290#issuecomment-921739924
Thanks for providing these very useful background information: without knowing exactly the context, I am already taking this approach in #287 , maybe not everywhere yet.
--
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: dev-unsubscribe@syncope.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org