Fix deprecations according latest SF

Use as much as possible SF API for proxies and their classes

Author: artembilan

spring-integration-core/src/main/java/org/springframework/integration/handler/support/MessagingMethodInvokerHelper.java

@@ -41,6 +41,7 @@
 import org.apache.commons.logging.LogFactory;
 
 import org.springframework.aop.framework.Advised;
+import org.springframework.aop.framework.AopProxyUtils;
 import org.springframework.aop.support.AopUtils;
 import org.springframework.beans.factory.BeanFactory;
 import org.springframework.beans.factory.config.BeanExpressionContext;
@@ -978,23 +979,20 @@ private String resolve(String value) {
 	}
 
 	private Class<?> getTargetClass(Object targetObject) {
-		Class<?> targetClass = targetObject.getClass();
-		if (AopUtils.isAopProxy(targetObject)) {
-			targetClass = AopUtils.getTargetClass(targetObject);
-			if (targetClass == targetObject.getClass()) {
-				try {
-					// Maybe a proxy with no target - e.g. gateway
-					Class<?>[] interfaces = ((Advised) targetObject).getProxiedInterfaces();
-					if (interfaces.length == 1) {
-						targetClass = interfaces[0];
-					}
-				}
-				catch (Exception e) {
-					LOGGER.debug("Exception trying to extract interface", e);
+		Class<?> targetClass = AopProxyUtils.ultimateTargetClass(targetObject);
+		if (targetClass == targetObject.getClass()) {
+			try {
+				// Maybe a proxy with no target - e.g. gateway
+				Class<?>[] interfaces = ((Advised) targetObject).getProxiedInterfaces();
+				if (interfaces.length == 1) {
+					targetClass = interfaces[0];
 				}
 			}
+			catch (Exception e) {
+				LOGGER.debug("Exception trying to extract interface", e);
+			}
 		}
-		else if (ClassUtils.isCglibProxyClass(targetClass) || targetClass.getSimpleName().contains("$MockitoMock$")) {
+		if (targetClass.getSimpleName().contains("$MockitoMock$")) {
 			Class<?> superClass = targetObject.getClass().getSuperclass();
 			if (!Object.class.equals(superClass)) {
 				targetClass = superClass;

spring-integration-core/src/main/java/org/springframework/integration/util/MessagingAnnotationUtils.java

@@ -21,7 +21,7 @@
 import java.util.List;
 import java.util.concurrent.atomic.AtomicReference;
 
-import org.springframework.aop.support.AopUtils;
+import org.springframework.aop.framework.AopProxyUtils;
 import org.springframework.core.annotation.AnnotatedElementUtils;
 import org.springframework.core.annotation.AnnotationUtils;
 import org.springframework.integration.annotation.EndpointId;
@@ -30,7 +30,6 @@
 import org.springframework.messaging.handler.annotation.Header;
 import org.springframework.messaging.handler.annotation.Headers;
 import org.springframework.messaging.handler.annotation.Payload;
-import org.springframework.util.ClassUtils;
 import org.springframework.util.ReflectionUtils;
 import org.springframework.util.StringUtils;
 
@@ -42,6 +41,7 @@
  * @author Gunnar Hillert
  * @author Soby Chacko
  * @author Artem Bilan
+ *
  * @since 4.0
  */
 public final class MessagingAnnotationUtils {
@@ -77,9 +77,9 @@ public static boolean hasValue(Object value) {
 	}
 
 	public static Method findAnnotatedMethod(Object target, final Class<? extends Annotation> annotationType) {
-		final AtomicReference<Method> reference = new AtomicReference<Method>();
+		final AtomicReference<Method> reference = new AtomicReference<>();
 
-		ReflectionUtils.doWithMethods(getTargetClass(target),
+		ReflectionUtils.doWithMethods(AopProxyUtils.ultimateTargetClass(target),
 				method -> reference.compareAndSet(null, method),
 				method -> ReflectionUtils.USER_DECLARED_METHODS.matches(method) &&
 						AnnotatedElementUtils.isAnnotated(method, annotationType.getName()));
@@ -129,20 +129,7 @@ public static String endpointIdValue(Method method) {
 		return endpointId != null ? endpointId.value() : null;
 	}
 
-	private static Class<?> getTargetClass(Object targetObject) {
-		Class<?> targetClass = targetObject.getClass();
-		if (AopUtils.isAopProxy(targetObject)) {
-			targetClass = AopUtils.getTargetClass(targetObject);
-		}
-		else if (ClassUtils.isCglibProxyClass(targetClass)) {
-			Class<?> superClass = targetObject.getClass().getSuperclass();
-			if (!Object.class.equals(superClass)) {
-				targetClass = superClass;
-			}
-		}
-		return targetClass;
+	private MessagingAnnotationUtils() {
 	}
 
-	private MessagingAnnotationUtils() { }
-
 }