Skip to content

Consider target-class proxy mode by default [SPR-14515] #19084

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

Closed
spring-projects-issues opened this issue Jul 26, 2016 · 2 comments
Closed

Consider target-class proxy mode by default [SPR-14515] #19084

spring-projects-issues opened this issue Jul 26, 2016 · 2 comments
Labels
in: core Issues in core modules (aop, beans, core, context, expression) status: declined A suggestion or change that we don't feel we should currently apply type: task A general task

Comments

@spring-projects-issues
Copy link
Collaborator

spring-projects-issues commented Jul 26, 2016

Juergen Hoeller opened SPR-14515 and commented

Following up on #19047, we should once again consider changing the default proxy mode to target-class (CGLIB)...


Affects: 5.0 M1

Issue Links:

@spring-projects-issues
Copy link
Collaborator Author

spring-projects-issues commented Aug 25, 2016

Willian Constâncio da Silva commented

Maybe #8346 is a related issue.

@jhoeller
Copy link
Contributor

Aside from Spring Boot's default target-class choice for high-level proxy functionality there, I don't see us ever changing this in the core framework facilities. At that level, this is effectively an independent setting for every proxy factory and would break many assumptions in scenarios that are hard to fix.

Also, in native images and similar scenarios, it actually turned out to be beneficial to aim for interface-based proxies wherever possible. Even for configuration classes, the "lite" mode (proxyBeanMethods=false) is rather recommendable for library components now, avoiding unnecessary class generation.

@jhoeller jhoeller added the status: declined A suggestion or change that we don't feel we should currently apply label Jul 14, 2023
@jhoeller jhoeller removed this from the General Backlog milestone Jul 14, 2023
@bclozel bclozel closed this as not planned Won't fix, can't repro, duplicate, stale Jul 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core Issues in core modules (aop, beans, core, context, expression) status: declined A suggestion or change that we don't feel we should currently apply type: task A general task
Projects
None yet
Development

No branches or pull requests

3 participants