AbstractAnnotationMethodEndpointMapping doesn't allow proxies [SWS-222]

[gregturn]

Ray Krueger opened SWS-222 and commented

The AbstractAnnotationMethodEndpointMapping looks up annotated method endpoints in the BeanPostProcessor::postProcessBeforeInitialization method. It then holds a reference to the bean inside of a "MethodEndpoint" instance.

The problem is that proxies get created AFTER the postProcessBeforeInitialization method. The lifecycle goes...

BeanPostProcessor::postProcessBeforeInitialization
AbstractAutoProxyCreator::createProxy
BeanPostProcessor::postProcessAfterInitialization

The AbstractAnnotationMethodEndpointMapping should implement postProcessAfterInitialization, not postProcessBeforeInitialization

---

Referenced from: commits 2e64fdf, 468b264

[gregturn]

Ray Krueger commented

Just to be clear...
I made this a critical as AOP is used heavily in Spring applications and there is no workaround available for this bug. Whereas bug #371 for example is critical because of the broken contract, but can be worked around by subclassing for a patch. Unfortunately, both postProcessBeforeInitialization and postProcessAfterInitialization are final in the AbstractAnnotationMethodEndpointMapping.

[gregturn]

Ray Krueger commented

Sorry, I meant to say "I made this a BLOCKER as..."

[gregturn]

Arjen Poutsma commented

Closing 1.0.2 issues.