From 9c0e9e44fb0327d1309b583fd9c228b9b98f4b4a Mon Sep 17 00:00:00 2001 From: Binary Wang Date: Sat, 10 Jun 2017 22:18:47 +0800 Subject: [PATCH] =?UTF-8?q?=E6=A3=80=E6=9F=A5=E7=AD=BE=E5=90=8D=E6=97=B6?= =?UTF-8?q?=EF=BC=8C=E5=A6=82=E6=9E=9C=E6=9C=89=E5=BC=82=E5=B8=B8=E6=89=93?= =?UTF-8?q?=E5=8D=B0=E5=87=BA=E6=97=A5=E5=BF=97?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../main/java/me/chanjar/weixin/common/util/crypto/SHA1.java | 2 +- .../me/chanjar/weixin/cp/api/impl/AbstractWxCpServiceImpl.java | 3 ++- .../me/chanjar/weixin/mp/api/impl/AbstractWxMpServiceImpl.java | 1 + 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/weixin-java-common/src/main/java/me/chanjar/weixin/common/util/crypto/SHA1.java b/weixin-java-common/src/main/java/me/chanjar/weixin/common/util/crypto/SHA1.java index 732d1bb85..3cdc57238 100644 --- a/weixin-java-common/src/main/java/me/chanjar/weixin/common/util/crypto/SHA1.java +++ b/weixin-java-common/src/main/java/me/chanjar/weixin/common/util/crypto/SHA1.java @@ -33,7 +33,7 @@ public class SHA1 { if (StringUtils.isAnyEmpty(arr)) { throw new IllegalArgumentException("非法请求参数，有部分参数为空 : " + Arrays.toString(arr)); } - + Arrays.sort(arr); StringBuilder sb = new StringBuilder(); for (int i = 0; i < arr.length; i++) { diff --git a/weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/impl/AbstractWxCpServiceImpl.java b/weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/impl/AbstractWxCpServiceImpl.java index b8eebe38e..463125b46 100644 --- a/weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/impl/AbstractWxCpServiceImpl.java +++ b/weixin-java-cp/src/main/java/me/chanjar/weixin/cp/api/impl/AbstractWxCpServiceImpl.java @@ -63,6 +63,7 @@ public abstract class AbstractWxCpServiceImpl implements WxCpService, Requ return SHA1.gen(this.configStorage.getToken(), timestamp, nonce, data) .equals(msgSignature); } catch (Exception e) { + this.log.error("Checking signature failed, and the reason is :" + e.getMessage()); return false; } } @@ -293,7 +294,7 @@ public abstract class AbstractWxCpServiceImpl implements WxCpService, Requ String responseContent = get(url, params); JsonElement tmpJsonElement = new JsonParser().parse(responseContent); return WxCpGsonBuilder.INSTANCE.create() - .fromJson( tmpJsonElement.getAsJsonObject().get("userlist"), + .fromJson(tmpJsonElement.getAsJsonObject().get("userlist"), new TypeToken>() { }.getType() ); diff --git a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/AbstractWxMpServiceImpl.java b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/AbstractWxMpServiceImpl.java index e1dfd83f4..85d307188 100644 --- a/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/AbstractWxMpServiceImpl.java +++ b/weixin-java-mp/src/main/java/me/chanjar/weixin/mp/api/impl/AbstractWxMpServiceImpl.java @@ -53,6 +53,7 @@ public abstract class AbstractWxMpServiceImpl implements WxMpService, Requ return SHA1.gen(this.getWxMpConfigStorage().getToken(), timestamp, nonce) .equals(signature); } catch (Exception e) { + this.log.error("Checking signature failed, and the reason is :" + e.getMessage()); return false; } }