From d964806405f21e4e2be4ed92106321f666c72cea Mon Sep 17 00:00:00 2001 From: parkjaehak Date: Mon, 10 Feb 2025 14:36:53 +0900 Subject: [PATCH 1/2] =?UTF-8?q?CLAP-358=20=20Fix:=20nickname=20=ED=8C=8C?= =?UTF-8?q?=EC=8B=B1=20=EB=B0=A9=EC=8B=9D=20=EB=B3=80=EA=B2=BD?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../clap/server/config/aop/LoggingAspect.java | 15 ++++----------- 1 file changed, 4 insertions(+), 11 deletions(-) diff --git a/src/main/java/clap/server/config/aop/LoggingAspect.java b/src/main/java/clap/server/config/aop/LoggingAspect.java index da7de978..90b745ae 100644 --- a/src/main/java/clap/server/config/aop/LoggingAspect.java +++ b/src/main/java/clap/server/config/aop/LoggingAspect.java @@ -7,7 +7,6 @@ import clap.server.common.annotation.log.LogType; import clap.server.exception.BaseException; import com.fasterxml.jackson.core.JsonProcessingException; -import com.fasterxml.jackson.databind.JsonNode; import com.fasterxml.jackson.databind.ObjectMapper; import jakarta.servlet.http.HttpServletRequest; import jakarta.servlet.http.HttpServletResponse; @@ -88,9 +87,9 @@ public Object logApiRequests(ProceedingJoinPoint joinPoint) throws Throwable { private void handleLoginLog(int statusCode, HttpServletRequest request, String customCode, LogStatus logStatus, Object result) throws JsonProcessingException { if (statusCode == HttpStatus.SC_OK) { - loggingPort.createAnonymousLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), getNicknameFromRequestBody(request)); + loggingPort.createAnonymousLog(request, statusCode, customCode, logStatus, result, getRequestBody(request), getNicknameFromParameter(request)); } else { - loggingPort.createLoginFailedLog(request, statusCode, customCode, logStatus, getRequestBody(request), getNicknameFromRequestBody(request)); + loggingPort.createLoginFailedLog(request, statusCode, customCode, logStatus, getRequestBody(request), getNicknameFromParameter(request)); } } @@ -102,14 +101,8 @@ private LogStatus getLogType(MethodSignature methodSignature) { } } - private String getNicknameFromRequestBody(HttpServletRequest request) { - try { - String requestBody = getRequestBody(request); - JsonNode jsonNode = objectMapper.readTree(requestBody); - return jsonNode.has("nickname") ? jsonNode.get("nickname").asText() : null; - } catch (Exception e) { - return null; - } + private String getNicknameFromParameter(HttpServletRequest request) { + return request.getParameter("nickname"); } private String getRequestBody(HttpServletRequest request) { From c0d26bf93e986ed04c751b89c72e673fca28d390 Mon Sep 17 00:00:00 2001 From: parkjaehak Date: Mon, 10 Feb 2025 14:39:35 +0900 Subject: [PATCH 2/2] =?UTF-8?q?CLAP-358=20=20Fix:=20=ED=9A=8C=EC=9B=90?= =?UTF-8?q?=EB=93=B1=EB=A1=9D=20=ED=85=8C=EC=8A=A4=ED=8A=B8=20=EC=9D=B4?= =?UTF-8?q?=EB=A6=84=20=EC=88=98=EC=A0=95?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- ...berCsvServiceTest.java => RegisterMemberCSVServiceTest.java} | 2 ++ 1 file changed, 2 insertions(+) rename src/test/java/clap/server/application/service/admin/{RegisterMemberCsvServiceTest.java => RegisterMemberCSVServiceTest.java} (97%) diff --git a/src/test/java/clap/server/application/service/admin/RegisterMemberCsvServiceTest.java b/src/test/java/clap/server/application/service/admin/RegisterMemberCSVServiceTest.java similarity index 97% rename from src/test/java/clap/server/application/service/admin/RegisterMemberCsvServiceTest.java rename to src/test/java/clap/server/application/service/admin/RegisterMemberCSVServiceTest.java index e28b309b..623a9384 100644 --- a/src/test/java/clap/server/application/service/admin/RegisterMemberCsvServiceTest.java +++ b/src/test/java/clap/server/application/service/admin/RegisterMemberCSVServiceTest.java @@ -3,6 +3,8 @@ import clap.server.application.port.inbound.domain.MemberService; import clap.server.application.port.outbound.member.CommandMemberPort; import clap.server.application.port.outbound.member.LoadMemberPort; +import clap.server.application.service.admin.CsvParseService; +import clap.server.application.service.admin.RegisterMemberCSVService; import clap.server.domain.model.member.Department; import clap.server.domain.model.member.Member; import clap.server.domain.model.member.MemberInfo;