检查签名时，如果有异常打印出日志

binarywang · binarywang · commit 9c0e9e44fb03 · 2017-06-10T22:18:47.000+08:00
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
@@ -33,7 +33,7 @@ public static String genWithAmple(String... arr) {
     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
@@ -63,6 +63,7 @@ public boolean checkSignature(String msgSignature, String timestamp, String nonc
       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 List<WxCpUser> userList(Integer departId, Boolean fetchChild, Integer sta
     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<List<WxCpUser>>() {
         }.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
@@ -53,6 +53,7 @@ public boolean checkSignature(String timestamp, String nonce, String signature)
       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;
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ public static String genWithAmple(String... arr) {`
`33`	`33`	`if (StringUtils.isAnyEmpty(arr)) {`
`34`	`34`	`throw new IllegalArgumentException("非法请求参数，有部分参数为空 : " + Arrays.toString(arr));`
`35`	`35`	`}`
`36`		`-`
	`36`	`+`
`37`	`37`	`Arrays.sort(arr);`
`38`	`38`	`StringBuilder sb = new StringBuilder();`
`39`	`39`	`for (int i = 0; i < arr.length; i++) {`
Original file line number	Diff line number	Diff line change
`@@ -53,6 +53,7 @@ public boolean checkSignature(String timestamp, String nonce, String signature)`
`53`	`53`	`return SHA1.gen(this.getWxMpConfigStorage().getToken(), timestamp, nonce)`
`54`	`54`	`.equals(signature);`
`55`	`55`	`} catch (Exception e) {`
	`56`	`+ this.log.error("Checking signature failed, and the reason is :" + e.getMessage());`
`56`	`57`	`return false;`
`57`	`58`	`}`
`58`	`59`	`}`