-
Notifications
You must be signed in to change notification settings - Fork 38.8k
AOP support for SPR-17223 & PR#1950 [SPR-17496] #22028
Copy link
Copy link
Closed
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)status: declinedA suggestion or change that we don't feel we should currently applyA suggestion or change that we don't feel we should currently applytype: enhancementA general enhancementA general enhancement
Metadata
Metadata
Assignees
Labels
in: coreIssues in core modules (aop, beans, core, context, expression)Issues in core modules (aop, beans, core, context, expression)status: declinedA suggestion or change that we don't feel we should currently applyA suggestion or change that we don't feel we should currently applytype: enhancementA general enhancementA general enhancement
Type
Fields
Give feedbackNo fields configured for issues without a type.
s-mop opened SPR-17496 and commented
In previous commit at PR#1950 we supported register RequestMapping on Interface which implement in parent class.And it works well in our production environment for this 7 weks. See #21756
And several days ago I realized that I can't register a AOP cutpoint for the scenario.
So i tried to changed the logic of org.springframework.aop.support.AopUtils.getMostSpecificMethod(Method, Class<?>)
Now this logic will try to find specific method in targetClass's interfaces first.Instead of only do that in targetClass.
I hope this commit can make my PR more helpful;)
Affects: 5.1 GA
Reference URL: c76f017
Issue Links:
@RequestMapping) in super classReferenced from: pull request #1950