Commit 332ebbb9 authored by lytscu's avatar lytscu
Browse files

spring mvc-annotation -3.x not work

parent 1cf06583

apm-sniffer/apm-sdk-plugin/spring-plugins/mvc-annotation-3.x-plugin/pom.xml

+2 −2
Original line number Diff line number Diff line
@@ -36,13 +36,13 @@ 
            <groupId>org.springframework</groupId>

            <artifactId>spring-core</artifactId>

            <version>3.2.18.RELEASE</version>

            <scope>compile</scope>

            <scope>provided</scope>

        </dependency>

        <dependency>

            <groupId>org.springframework</groupId>

            <artifactId>spring-webmvc</artifactId>

            <version>3.2.18.RELEASE</version>

            <scope>compile</scope>

            <scope>provided</scope>

        </dependency>

        <dependency>

            <groupId>javax.servlet</groupId>

apm-sniffer/apm-sdk-plugin/spring-plugins/mvc-annotation-4.x-plugin/pom.xml

+2 −2
Original line number Diff line number Diff line
@@ -36,13 +36,13 @@ 
            <groupId>org.springframework</groupId>

            <artifactId>spring-core</artifactId>

            <version>4.3.10.RELEASE</version>

            <scope>compile</scope>

            <scope>provided</scope>

        </dependency>

        <dependency>

            <groupId>org.springframework</groupId>

            <artifactId>spring-webmvc</artifactId>

            <version>4.3.8.RELEASE</version>

            <scope>compile</scope>

            <scope>provided</scope>

        </dependency>

        <dependency>

            <groupId>javax.servlet</groupId>

apm-sniffer/apm-sdk-plugin/spring-plugins/mvc-annotation-4.x-plugin/src/main/java/org/skywalking/apm/plugin/spring/mvc/v4/define/AbstractControllerInstrumentation.java

+5 −5
Original line number Diff line number Diff line
@@ -35,10 +35,10 @@ import static org.skywalking.apm.agent.core.plugin.match.ClassAnnotationMatch.by 
 * <code>org.springframework.web.bind.annotation.RequestMapping</code> that class has

 * <code>org.springframework.stereotype.Controller</code> annotation.

 *

 * <code>org.skywalking.apm.plugin.spring.mvc.ControllerConstructorInterceptor</code> set the controller base path to

 * <code>org.skywalking.apm.plugin.spring.mvc.v4.ControllerConstructorInterceptor</code> set the controller base path to

 * dynamic field before execute constructor.

 *

 * <code>org.skywalking.apm.plugin.spring.mvc.RequestMappingMethodInterceptor</code> get the request path from

 * <code>org.skywalking.apm.plugin.spring.mvc.v4.RequestMappingMethodInterceptor</code> get the request path from

 * dynamic field first, if not found, <code>RequestMappingMethodInterceptor</code> generate request path  that

 * combine the path value of current annotation on current method and the base path and set the new path to the dynamic

 * filed
@@ -57,7 +57,7 @@ public abstract class AbstractControllerInstrumentation extends ClassInstanceMet 


                @Override

                public String getConstructorInterceptor() {

                    return "org.skywalking.apm.plugin.spring.mvc.ControllerConstructorInterceptor";

                    return "org.skywalking.apm.plugin.spring.mvc.v4.ControllerConstructorInterceptor";

                }

            }

        };
@@ -74,7 +74,7 @@ public abstract class AbstractControllerInstrumentation extends ClassInstanceMet 


                @Override

                public String getMethodsInterceptor() {

                    return "org.skywalking.apm.plugin.spring.mvc.RequestMappingMethodInterceptor";

                    return "org.skywalking.apm.plugin.spring.mvc.v4.RequestMappingMethodInterceptor";

                }



                @Override
@@ -94,7 +94,7 @@ public abstract class AbstractControllerInstrumentation extends ClassInstanceMet 


                @Override

                public String getMethodsInterceptor() {

                    return "org.skywalking.apm.plugin.spring.mvc.RestMappingMethodInterceptor";

                    return "org.skywalking.apm.plugin.spring.mvc.v4.RestMappingMethodInterceptor";

                }



                @Override