diff --git a/broker/src/main/java/com/xiaoleilu/loServer/action/admin/AdminAction.java b/broker/src/main/java/com/xiaoleilu/loServer/action/admin/AdminAction.java index 4f84956a..807e4d03 100644 --- a/broker/src/main/java/com/xiaoleilu/loServer/action/admin/AdminAction.java +++ b/broker/src/main/java/com/xiaoleilu/loServer/action/admin/AdminAction.java @@ -59,7 +59,7 @@ abstract public class AdminAction extends Action { } if (StringUtil.isNullOrEmpty(nonce) || StringUtil.isNullOrEmpty(timestamp) || StringUtil.isNullOrEmpty(sign)) { - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_API_NOT_SIGNED; } Long ts; @@ -68,7 +68,7 @@ abstract public class AdminAction extends Action { } catch (Exception e) { e.printStackTrace(); Utility.printExecption(LOG, e); - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_API_NOT_SIGNED; } if (!NO_CHECK_TIME && System.currentTimeMillis() - ts > 2 * 60 * 60 * 1000) { diff --git a/broker/src/main/java/com/xiaoleilu/loServer/action/channel/ChannelAction.java b/broker/src/main/java/com/xiaoleilu/loServer/action/channel/ChannelAction.java index 61b92b68..978dc27d 100644 --- a/broker/src/main/java/com/xiaoleilu/loServer/action/channel/ChannelAction.java +++ b/broker/src/main/java/com/xiaoleilu/loServer/action/channel/ChannelAction.java @@ -51,7 +51,7 @@ abstract public class ChannelAction extends Action { } if (StringUtil.isNullOrEmpty(nonce) || StringUtil.isNullOrEmpty(timestamp) || StringUtil.isNullOrEmpty(sign) || StringUtil.isNullOrEmpty(cid)) { - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_API_NOT_SIGNED; } if (!mLimitCounter.isGranted(cid)) { @@ -64,7 +64,7 @@ abstract public class ChannelAction extends Action { } catch (Exception e) { e.printStackTrace(); Utility.printExecption(LOG, e, IMExceptionEvent.EventType.CHANNEL_API_Exception); - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_API_NOT_SIGNED; } if (System.currentTimeMillis() - ts > 2 * 60 * 60 * 1000) { diff --git a/broker/src/main/java/com/xiaoleilu/loServer/action/robot/RobotAction.java b/broker/src/main/java/com/xiaoleilu/loServer/action/robot/RobotAction.java index 3dbf5ca6..d608e158 100644 --- a/broker/src/main/java/com/xiaoleilu/loServer/action/robot/RobotAction.java +++ b/broker/src/main/java/com/xiaoleilu/loServer/action/robot/RobotAction.java @@ -55,7 +55,7 @@ abstract public class RobotAction extends Action { } if (StringUtil.isNullOrEmpty(nonce) || StringUtil.isNullOrEmpty(timestamp) || StringUtil.isNullOrEmpty(sign) || StringUtil.isNullOrEmpty(rid)) { - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_API_NOT_SIGNED; } if (!mLimitCounter.isGranted(rid)) { @@ -68,7 +68,7 @@ abstract public class RobotAction extends Action { } catch (Exception e) { e.printStackTrace(); Utility.printExecption(LOG, e, IMExceptionEvent.EventType.ROBOT_API_Exception); - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_API_NOT_SIGNED; } if (System.currentTimeMillis() - ts > 2 * 60 * 60 * 1000) { diff --git a/common/src/main/java/cn/wildfirechat/common/ErrorCode.java b/common/src/main/java/cn/wildfirechat/common/ErrorCode.java index 7e697cde..6b4905db 100755 --- a/common/src/main/java/cn/wildfirechat/common/ErrorCode.java +++ b/common/src/main/java/cn/wildfirechat/common/ErrorCode.java @@ -42,6 +42,7 @@ public enum ErrorCode { ERROR_CODE_RECALL_TIME_EXPIRED(24, "time expired"), + ERROR_CODE_API_NOT_SIGNED(239, "api not signed or sign parameter not completion"), ERROR_CODE_GROUP_EXCEED_MAX_MEMBER_COUNT(240, "group exceed max member count"), ERROR_CODE_GROUP_MUTED(241, "group is muted"), ERROR_CODE_SENSITIVE_MATCHED(242, "sensitive matched"),