Подтвердить что ты не робот

Смешивание прокси-серверов JDK и CGLIB в Spring

У меня есть приложение, работающее с Spring, и я использую AOP в некоторых местах. Поскольку я хочу использовать аннотацию @Transactional на уровне интерфейса, я должен разрешить Spring создавать прокси-серверы JDK. Итак, я не устанавливаю для свойства proxy-target-class значение true. С другой стороны, я не хочу создавать интерфейс для каждого отдельного класса, о котором я хочу посоветовать: если интерфейс просто не имеет смысла, я хочу иметь только реализацию, а Spring должен создать прокси CGLIB.

Все работало отлично, как я описал. Но я хотел, чтобы некоторые другие аннотации (созданные мной) вошли в интерфейсы и были "унаследованы" классами реализации (как и @Transactional). Оказывается, я не могу этого сделать со встроенной поддержкой AOP в Spring (по крайней мере, я не мог понять, как это сделать после некоторого исследования. Аннотации в интерфейсе не видны в классе реализации, и, следовательно, этот класс не рекомендуется).

Итак, я решил реализовать свой собственный pointcut и перехватчик, позволяя другим аннотациям методов идти на интерфейсы. В принципе, мой pointcut ищет аннотацию к методу и, пока не найден, в том же методе (одно имя и типы параметров) интерфейсов, которые реализует класс или его суперклассы.

Проблема заключается в следующем: когда я объявляю DefaultAdvisorAutoProxyCreator bean, который будет правильно применять этот pointcut/interceptor, поведение советов классов без интерфейсов нарушается. Видимо, что-то пошло не так, и Spring пытается проксировать мои классы дважды, один раз с CGLIB, а затем с JDK.

Это мой файл конфигурации:

<beans xmlns="http://www.springframework.org/schema/beans"
    xmlns:context="http://www.springframework.org/schema/context"
    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:tx="http://www.springframework.org/schema/tx"
    xmlns:aop="http://www.springframework.org/schema/aop" xmlns:task="http://www.springframework.org/schema/task"
    xsi:schemaLocation="
    http://www.springframework.org/schema/beans http://www.springframework.org/schema/beans/spring-beans-3.0.xsd
        http://www.springframework.org/schema/context http://www.springframework.org/schema/context/spring-context-3.0.xsd
        http://www.springframework.org/schema/tx http://www.springframework.org/schema/tx/spring-tx-3.0.xsd
        http://www.springframework.org/schema/aop http://www.springframework.org/schema/aop/spring-aop-3.0.xsd
    http://www.springframework.org/schema/task http://www.springframework.org/schema/task/spring-task-3.0.xsd">

    <!-- Activates various annotations to be detected in bean classes: Spring 
        @Required and @Autowired, as well as JSR 250 @Resource. -->
    <context:annotation-config />

    <context:component-scan base-package="mypackage" />

    <!-- Instruct Spring to perform declarative transaction management automatically 
        on annotated classes. -->
    <tx:annotation-driven transaction-manager="transactionManager" />

    <bean class="org.springframework.aop.framework.autoproxy.DefaultAdvisorAutoProxyCreator" />

    <bean id="logger.advisor" class="org.springframework.aop.support.DefaultPointcutAdvisor">
        <constructor-arg>
            <bean class="mypackage.MethodAnnotationPointcut">
                <constructor-arg value="mypackage.Trace" />
            </bean>
        </constructor-arg>
        <constructor-arg>
            <bean class="mypackage.TraceInterceptor" />
        </constructor-arg>
    </bean>
</beans>

Это класс, который я хочу проксировать, без интерфейсов:

@Component
public class ServiceExecutorImpl
{
    @Transactional
    public Object execute(...)
    {
        ...
    }
}

Когда я пытаюсь выполнить его автоопределение в другом bean, например:

public class BaseService {
   @Autowired
   private ServiceExecutorImpl serviceExecutorImpl;

   ...
}

Я получаю следующее исключение:

java.lang.IllegalArgumentException: Can not set mypackage.ServiceExecutorImpl field mypackage.BaseService.serviceExecutor to $Proxy26

Это несколько строк вывода Spring:

13:51:12,672 [main] DEBUG [org.springframework.aop.framework.Cglib2AopProxy] - Creating CGLIB2 proxy: target source is SingletonTargetSource for target object [[email protected]]
...
13:51:12,782 [main] DEBUG [org.springframework.aop.framework.autoproxy.DefaultAdvisorAutoProxyCreator] - Creating implicit proxy for bean 'serviceExecutorImpl' with 0 common interceptors and 1 specific interceptors
13:51:12,783 [main] DEBUG [org.springframework.aop.framework.JdkDynamicAopProxy] - Creating JDK dynamic proxy: target source is SingletonTargetSource for target object [[email protected]]

Я мог бы предоставить полную информацию, если кто-то подумает, что это поможет. Я не знаю, почему Spring пытается "удвоить прокси" мой класс, и почему это происходит, когда я объявляю DefaultAdvisorAutoProxyCreator bean.

Я боролся с этим в течение некоторого времени, и любая помощь или идеи были бы очень оценены.

EDIT:

Это мой исходный код для перехватчиков, как было запрошено. Он в основном регистрирует выполнение метода (только методы, аннотированные с помощью @Trace, перехватываются). Если метод аннотируется с помощью @Trace (false), ведение журнала приостанавливается до тех пор, пока метод не вернется.

public class TraceInterceptor
    implements
        MethodInterceptor
{

    @Override
    public Object invoke(
        MethodInvocation invocation )
        throws Throwable
    {
        if( ThreadExecutionContext.getCurrentContext().isLogSuspended() ) {
            return invocation.proceed();
        }

        Method method = AopUtils.getMostSpecificMethod( invocation.getMethod(),
            invocation.getThis().getClass() );
        Trace traceAnnotation = method.getAnnotation( Trace.class );

        if( traceAnnotation != null && traceAnnotation.value() == false ) {
            ThreadExecutionContext.getCurrentContext().suspendLogging();
            Object result = invocation.proceed();
            ThreadExecutionContext.getCurrentContext().resumeLogging();
            return result;
        }

        ThreadExecutionContext.startNestedLevel();
        SimpleDateFormat dateFormat = new SimpleDateFormat( "dd/MM/yyyy - HH:mm:ss.SSS" );
        Logger.log( "Timestamp: " + dateFormat.format( new Date() ) );

        String toString = invocation.getThis().toString();
        Logger.log( "Class: " + toString.substring( 0, toString.lastIndexOf( '@' ) ) );

        Logger.log( "Method: " + getMethodName( method ) );
        Logger.log( "Parameters: " );
        for( Object arg : invocation.getArguments() ) {
            Logger.log( arg );
        }

        long before = System.currentTimeMillis();
        try {
            Object result = invocation.proceed();
            Logger.log( "Return: " );
            Logger.log( result );
            return result;
        } finally {
            long after = System.currentTimeMillis();
            Logger.log( "Total execution time (ms): " + ( after - before ) );
            ThreadExecutionContext.endNestedLevel();
        }
    }

    // Just formats a method name, with parameter and return types
    private String getMethodName(
        Method method )
    {
        StringBuffer methodName = new StringBuffer( method.getReturnType().getSimpleName() + " "
            + method.getName() + "(" );
        Class<?>[] parameterTypes = method.getParameterTypes();

        if( parameterTypes.length == 0 ) {
            methodName.append( ")" );
        } else {
            int index;
            for( index = 0; index < ( parameterTypes.length - 1 ); index++ ) {
                methodName.append( parameterTypes[ index ].getSimpleName() + ", " );
            }
            methodName.append( parameterTypes[ index ].getSimpleName() + ")" );
        }
        return methodName.toString();
    }
}

Спасибо!

4b9b3361

Ответ 1

Я нашел решение с использованием "scoped-proxy", предложенного Божо.

Поскольку я использую почти только аннотации, мой класс ServiceExecutor теперь выглядит следующим образом:

@Component
@Scope( proxyMode = ScopedProxyMode.TARGET_CLASS )
public class ServiceExecutor
{
    @Transactional
    public Object execute(...)
    {
        ...
    }
}

До сих пор все сидения работают нормально. Я не знаю, почему я должен явно указывать Spring этот класс должен быть проксирован с использованием CGLIB, так как он не реализует никакого интерфейса. Может быть, это ошибка, я не знаю.

Спасибо, Божо.

Ответ 2

Что-то здесь не соответствует - если это $ProxyXX, это означает, что есть интерфейс. Убедитесь, что нет интерфейса. Некоторые другие примечания:

  • в вашем pointcut вы можете проверить, является ли целевой объект уже прокси-сервером с помощью (x instanceof Advised), затем вы можете использовать Advised

  • вы можете использовать <aop:scoped-proxy /> для определения стратегии прокси-сервера per bean