From 963b07b800a3e3cf583dbdbdfc7c972d2b87c3e5 Mon Sep 17 00:00:00 2001
From: ascrutae <ascrutae@gmail.com>
Date: Wed, 14 Dec 2016 17:44:15 +0800
Subject: [PATCH] =?UTF-8?q?=E4=BF=AE=E5=A4=8D=E9=83=A8=E5=88=86=E9=97=AE?=
 =?UTF-8?q?=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../skywalking/plugin/bytebuddy/ArgumentTypeNameMatch.java  | 6 +++++-
 .../skywalking-sdk-plugin/jedis-2.x-plugin/pom.xml          | 2 +-
 .../plugin/jedis/v2/define/JedisPluginDefine.java           | 4 +++-
 3 files changed, 9 insertions(+), 3 deletions(-)

diff --git a/skywalking-sniffer/skywalking-api/src/main/java/com/a/eye/skywalking/plugin/bytebuddy/ArgumentTypeNameMatch.java b/skywalking-sniffer/skywalking-api/src/main/java/com/a/eye/skywalking/plugin/bytebuddy/ArgumentTypeNameMatch.java
index fcd197c79..8106f5437 100644
--- a/skywalking-sniffer/skywalking-api/src/main/java/com/a/eye/skywalking/plugin/bytebuddy/ArgumentTypeNameMatch.java
+++ b/skywalking-sniffer/skywalking-api/src/main/java/com/a/eye/skywalking/plugin/bytebuddy/ArgumentTypeNameMatch.java
@@ -18,6 +18,10 @@ public class ArgumentTypeNameMatch implements ElementMatcher<MethodDescription>
 
     @Override
     public boolean matches(MethodDescription target) {
-        return target.getParameters().get(index).getType().asErasure().getName().equals(argumentTypeName);
+        if (target.getParameters().size() > index) {
+            return target.getParameters().get(index).getType().asErasure().getName().equals(argumentTypeName);
+        }
+
+        return false;
     }
 }
diff --git a/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/pom.xml b/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/pom.xml
index fce4a7a68..c6c75fc69 100644
--- a/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/pom.xml
+++ b/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/pom.xml
@@ -23,7 +23,7 @@
 		<dependency>
 			<groupId>redis.clients</groupId>
 			<artifactId>jedis</artifactId>
-			<version>2.8.1</version>
+			<version>2.9.0</version>
 			<scope>provided</scope>
 		</dependency>
 
diff --git a/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/src/main/java/com/a/eye/skywalking/plugin/jedis/v2/define/JedisPluginDefine.java b/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/src/main/java/com/a/eye/skywalking/plugin/jedis/v2/define/JedisPluginDefine.java
index 9d262f002..9a5b88d30 100644
--- a/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/src/main/java/com/a/eye/skywalking/plugin/jedis/v2/define/JedisPluginDefine.java
+++ b/skywalking-sniffer/skywalking-sdk-plugin/jedis-2.x-plugin/src/main/java/com/a/eye/skywalking/plugin/jedis/v2/define/JedisPluginDefine.java
@@ -11,6 +11,8 @@ import com.a.eye.skywalking.plugin.interceptor.matcher.SimpleMethodMatcher;
 import net.bytebuddy.description.method.MethodDescription;
 import net.bytebuddy.matcher.ElementMatcher;
 
+import java.net.URI;
+
 import static net.bytebuddy.matcher.ElementMatchers.not;
 import static net.bytebuddy.matcher.ElementMatchers.takesArgument;
 
@@ -46,7 +48,7 @@ public class JedisPluginDefine extends ClassInstanceMethodsEnhancePluginDefine {
         }, new ConstructorInterceptPoint() {
             @Override
             public ElementMatcher<MethodDescription> getConstructorMatcher() {
-                return takesArgument(0, String.class);
+                return takesArgument(0, URI.class);
             }
 
             @Override
-- 
GitLab