From d901707691711ab2b9374a11f4eba62bf9fdd3db Mon Sep 17 00:00:00 2001 From: heavyrain2012 Date: Tue, 28 Dec 2021 19:22:55 +0800 Subject: [PATCH] add log --- .../com/xiaoleilu/loServer/action/channel/ChannelAction.java | 3 ++- common/src/main/java/cn/wildfirechat/common/ErrorCode.java | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) 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 1e8541ba..2a7ba4d8 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 @@ -70,6 +70,7 @@ abstract public class ChannelAction extends Action { try { ts = Long.parseLong(timestamp); } catch (Exception e) { + LOG.error("Channel timestamp:{} invalid", timestamp); e.printStackTrace(); Utility.printExecption(LOG, e, IMExceptionEvent.EventType.CHANNEL_API_Exception); return ErrorCode.ERROR_CODE_API_NOT_SIGNED; @@ -81,7 +82,7 @@ abstract public class ChannelAction extends Action { channelInfo = messagesStore.getChannelInfo(cid); if (channelInfo == null) { - return ErrorCode.INVALID_PARAMETER; + return ErrorCode.ERROR_CODE_CHANNEL_NO_EXIST; } if (StringUtil.isNullOrEmpty(channelInfo.getSecret())) { diff --git a/common/src/main/java/cn/wildfirechat/common/ErrorCode.java b/common/src/main/java/cn/wildfirechat/common/ErrorCode.java index 4f666158..8f5b01b5 100755 --- a/common/src/main/java/cn/wildfirechat/common/ErrorCode.java +++ b/common/src/main/java/cn/wildfirechat/common/ErrorCode.java @@ -49,6 +49,7 @@ public enum ErrorCode { ERROR_CODE_WS_NOT_CONFIGURED_CORRECTLY(28, "ws not configured correctly"), ERROR_CODE_CONFERENCE_ROOM_NOT_EXIST(60, "conference room not exist"), + ERROR_CODE_CHANNEL_NO_EXIST(236, "channel no exist"), ERROR_CODE_CHANNEL_NO_SECRET(237, "channel no secret"), ERROR_CODE_USER_NOT_PREPARED(238, "user not prepared"), ERROR_CODE_API_NOT_SIGNED(239, "api not signed or sign parameter not completion"),