한성재 sprint7 by seonghj · Pull Request #159 · codeit-bootcamp-spring/9-sprint-mission

seonghj · 2026-04-05T01:15:06Z

요구사항

기본

프로파일 기반 설정 관리
로그 관리
예외 처리 고도화
유효성 검사
Actuator
단위 테스트
슬라이스 테스트
통합 테스트

심화

MDC를 활용한 로깅 고도화
Spring Boot Admin을 활용한 메트릭 가시화
테스트 커버리지 관리 (60% 달성)

주요 변경사항

스크린샷

멘토에게

셀프 코드 리뷰를 통해 질문 이어가겠습니다.

기본 요구사함 프로파일 기반 설정 관리에 맞춘 프로파일 추가

Spring Validation 의존성을 추가

주요 Request에 제약 조건 관련 어노테이션 추가, GlobalExceptionHandler에 handleMethodArgumentNotValidException 추가

joonfluence

리뷰 요약

Sprint 7 요구사항(프로필 기반 설정, 로깅, 예외 처리, Validation, Actuator, 테스트)을 잘 구현했습니다. 전반적으로 구조가 깔끔하게 잡혀 있으나, 아래 이슈들을 확인해주세요.

joonfluence

각 코드 위치에 인라인 코멘트로 리뷰를 작성했습니다. Critical → Major → Minor 순으로 확인해주세요.

joonfluence · 2026-04-05T16:16:20Z

+    @ExceptionHandler(DiscodeitException.class)
+    public ResponseEntity<ErrorResponse> handleDiscodeitException(DiscodeitException e){
+        HttpStatus status = HttpStatus.INTERNAL_SERVER_ERROR;
+        ErrorResponse response = ErrorResponse.of(e, status.value());


[Critical] DiscodeitException fallback 핸들러가 항상 500 반환

새로운 커스텀 예외를 추가할 때 전용 핸들러를 빠뜨리면 이 fallback에 걸려 항상 500 Internal Server Error가 반환됩니다.

ErrorCode에 HTTP status를 매핑하거나, DiscodeitException에 getHttpStatus() 메서드를 추가하세요.

// ErrorCode에 status 추가 예시 USER_NOT_FOUND(2001, "존재하지 않는 사용자입니다.", HttpStatus.NOT_FOUND), // fallback 핸들러에서 활용 HttpStatus status = e.getErrorCode().getHttpStatus();

joonfluence · 2026-04-05T16:16:20Z

+    // Message
+    @ExceptionHandler(MessageNotFoundException.class)
+    public ResponseEntity<ErrorResponse> handleChannelNotFoundException(MessageNotFoundException e){
+        HttpStatus status = HttpStatus.NOT_FOUND;


[Minor] 메서드 이름 중복

handleChannelNotFoundException이라는 이름의 핸들러가 ChannelNotFoundException과 MessageNotFoundException 두 곳에 정의되어 있습니다. 컴파일은 되지만 혼란을 줍니다.

// 변경 public ResponseEntity<ErrorResponse> handleMessageNotFoundException(MessageNotFoundException e) { ... }

joonfluence · 2026-04-05T16:16:20Z

        throws IOException {
-
+        log.info("사용자 정보 수정 요청 수신: username={}, userId={}",request, userId);
        Optional<BinaryContentCreateRequest> binaryContentCreateRequest = Optional.empty();


[Critical] 로그에 비밀번호 노출 위험

request 객체를 통째로 로깅하면 UserUpdateRequest의 newPassword 필드가 로그에 평문으로 기록될 수 있습니다.

// 수정 log.info("사용자 정보 수정 요청 수신: username={}, userId={}", request.newUsername(), userId);

joonfluence · 2026-04-05T16:16:20Z

+                });

        if (!user.getPassword().equals(request.password())){
+            log.warn("로그인 실패: 비밀번호 불일치 - username={}", request.username());


[Critical] 비밀번호 평문 비교

비밀번호를 평문으로 저장·비교하고 있습니다. 학습 프로젝트라도 PasswordEncoder(BCrypt 등) 사용을 권장합니다.

// 예시 if (!passwordEncoder.matches(request.password(), user.getPassword())) { throw new AuthenticationException(...); }

joonfluence · 2026-04-05T16:16:20Z

 import org.springframework.data.annotation.CreatedDate;
 import org.springframework.data.jpa.domain.support.AuditingEntityListener;
+import org.springframework.data.jpa.repository.config.EnableJpaAuditing;



[Major] 미사용 import

@EnableJpaAuditing은 JpaConfig로 이동했는데, 여기서는 사용하지 않는 import가 추가되었습니다. 제거해주세요.

joonfluence · 2026-04-05T16:16:20Z


+
+import jakarta.validation.constraints.*;
+


[Major] 와일드카드 import 사용

jakarta.validation.constraints.* 대신 필요한 애노테이션만 명시적으로 import하세요. 코드 가독성이 높아지고 IDE 지원도 더 원활합니다.

import jakarta.validation.constraints.Email; import jakarta.validation.constraints.NotBlank; import jakarta.validation.constraints.Size;

joonfluence · 2026-04-05T16:16:20Z

 discodeit.iml
-/storage/
+/storage/
+


[Minor] /storage/ 항목 중복

/storage/가 이 파일에 두 번 정의되어 있습니다. 하나를 제거해주세요.

joonfluence · 2026-04-05T16:16:20Z

+
+  // MESSAGE
+  MESSAGE_NOT_FOUND(4001, "메시지가 존재하지 않습니다")
+  ;


[Minor] 마침표 누락으로 일관성 깨짐

다른 ErrorCode 항목은 메시지 끝에 마침표가 있는데 MESSAGE_NOT_FOUND만 없습니다.

MESSAGE_NOT_FOUND(4001, "메시지가 존재하지 않습니다.") // 마침표 추가

joonfluence · 2026-04-05T16:16:20Z

+plugins {
+    id 'java'
+    id 'org.springframework.boot' version '4.0.5'
+    id 'io.spring.dependency-management' version '1.1.7'


[Major] Spring Boot 버전 오류

org.springframework.boot version '4.0.5'는 존재하지 않는 버전입니다. Spring Boot는 현재 3.x 라인입니다. discodeit 모듈(3.5.10 — 이것도 확인 필요)과 동일한 버전으로 맞춰주세요.

또한 springBootAdminVersion도 "4.0.2"로 설정되어 있는데, 이 버전도 존재하지 않습니다.

joonfluence · 2026-04-05T16:16:20Z

@@ -0,0 +1,163 @@
+package com.sprint.mission.discodeit.integraton;
+import static org.springframework.security.test.web.servlet.request.SecurityMockMvcRequestPostProcessors.csrf;


[Minor] 패키지 이름 오타

integraton은 integration의 오타입니다. 패키지 이름과 디렉토리를 수정해주세요.

// 수정 package com.sprint.mission.discodeit.integration;

joonfluence · 2026-04-06T00:33:43Z

+    // custom exception
+    @ExceptionHandler(DiscodeitException.class)
+    public ResponseEntity<ErrorResponse> handleDiscodeitException(DiscodeitException e){
+        HttpStatus status = HttpStatus.INTERNAL_SERVER_ERROR;


[Critical] DiscodeitException fallback 핸들러가 항상 500 반환

새로운 커스텀 예외를 추가할 때 전용 핸들러를 빠뜨리면 이 fallback에 걸려 항상 500 Internal Server Error가 반환됩니다.

ErrorCode에 HTTP status를 매핑하거나, DiscodeitException에 getHttpStatus() 메서드를 추가하세요.

// ErrorCode에 status 추가 예시 USER_NOT_FOUND(2001, "존재하지 않는 사용자입니다.", HttpStatus.NOT_FOUND), // fallback 핸들러에서 활용 HttpStatus status = e.getErrorCode().getHttpStatus();

joonfluence · 2026-04-06T00:33:44Z

+
+    // Message
+    @ExceptionHandler(MessageNotFoundException.class)
+    public ResponseEntity<ErrorResponse> handleChannelNotFoundException(MessageNotFoundException e){


[Minor] 메서드 이름 중복

handleChannelNotFoundException이라는 이름의 핸들러가 ChannelNotFoundException과 MessageNotFoundException 두 곳에 정의되어 있습니다. 컴파일은 되지만 혼란을 줍니다.

// 변경 public ResponseEntity<ErrorResponse> handleMessageNotFoundException(MessageNotFoundException e) { ... }

joonfluence · 2026-04-06T00:33:45Z

        , @RequestPart(value = "profile", required = false) MultipartFile imageFile)
        throws IOException {
-
+        log.info("사용자 정보 수정 요청 수신: username={}, userId={}",request, userId);


[Critical] 로그에 비밀번호 노출 위험

request 객체를 통째로 로깅하면 UserUpdateRequest의 newPassword 필드가 로그에 평문으로 기록될 수 있습니다.

// 수정 log.info("사용자 정보 수신: username={}, userId={}", request.newUsername(), userId);

joonfluence · 2026-04-06T00:33:46Z

+                    return new AuthenticationException("Not Exist User - userName: " + request.username());
+                });

        if (!user.getPassword().equals(request.password())){


[Critical] 비밀번호 평문 비교

비밀번호를 평문으로 저장·비교하고 있습니다. 학습 프로젝트라도 PasswordEncoder(BCrypt 등) 사용을 권장합니다.

// 예시 if (!passwordEncoder.matches(request.password(), user.getPassword())) { throw new AuthenticationException(...); }

joonfluence · 2026-04-06T00:33:47Z


 import org.springframework.data.annotation.CreatedDate;
 import org.springframework.data.jpa.domain.support.AuditingEntityListener;
+import org.springframework.data.jpa.repository.config.EnableJpaAuditing;


[Major] 미사용 import

@EnableJpaAuditing은 JpaConfig로 이동했습니다. 이 파일에서는 사용하지 않으므로 제거해주세요.

joonfluence · 2026-04-06T00:33:48Z

 import jakarta.persistence.criteria.CriteriaBuilder.In;
 import java.util.NoSuchElementException;
+import lombok.extern.slf4j.Slf4j;
+import org.springframework.data.jpa.repository.config.EnableJpaAuditing;


[Major] Service 클래스에 미사용 import

@EnableJpaAuditing은 @Configuration 클래스(JpaConfig)에서만 사용해야 합니다. 이 Service에서는 사용되지 않으므로 제거해주세요.

joonfluence · 2026-04-06T00:33:49Z

+            log.warn("채널 삭제 실패: 존재하지 않는 채널 ID={}", id);
+            return new ChannelNotFoundException(id);
+        });
+        log.info("채널 삭제 완료: channelId={}", id);


[Minor] 로그 순서 오류 — 삭제 전에 "완료" 로그가 찍힘

현재 channelRepository.delete(channel) 호출 전에 "채널 삭제 완료" 로그가 기록됩니다. DB 작업 실패 시 로그와 실제 상태가 불일치합니다.

channelRepository.delete(channel); log.info("채널 삭제 완료: channelId={}", id); // 이 위치로 이동

joonfluence · 2026-04-06T00:33:50Z

 import java.util.UUID;

 public record MessageCreateRequest(
+    @Size(max = 100, message = "메시지 내용은 최대 100자입니다.")


[Major] content 필드에 @NotBlank 누락

@Size(max = 100)만으로는 null 또는 빈 문자열("")을 막지 못합니다. 빈 메시지 전송을 방지하려면 @NotBlank를 함께 사용해야 합니다.

@NotBlank(message = "메시지 내용은 필수입니다.") @Size(max = 100, message = "메시지 내용은 최대 100자입니다.") String content,

joonfluence · 2026-04-06T00:33:51Z

 package com.sprint.mission.discodeit.dto.request;

+
+import jakarta.validation.constraints.*;


[Major] 와일드카드 import 사용

jakarta.validation.constraints.* 대신 필요한 애노테이션만 명시적으로 import하세요.

import jakarta.validation.constraints.Email; import jakarta.validation.constraints.NotBlank; import jakarta.validation.constraints.Size;

joonfluence · 2026-04-06T00:33:52Z

+  PRIVATE_CHANNEL_UPDATE(3002, "PRIVATE 채널은 수정할 수 없습니다."),
+
+  // MESSAGE
+  MESSAGE_NOT_FOUND(4001, "메시지가 존재하지 않습니다")


[Minor] 마침표 누락으로 일관성 깨짐

다른 ErrorCode 항목은 메시지 끝에 마침표가 있는데 MESSAGE_NOT_FOUND만 없습니다.

MESSAGE_NOT_FOUND(4001, "메시지가 존재하지 않습니다.") // 마침표 추가

joonfluence · 2026-04-06T00:33:53Z

@@ -0,0 +1,39 @@
+plugins {
+    id 'java'
+    id 'org.springframework.boot' version '4.0.5'


[Major] 존재하지 않는 Spring Boot 버전

org.springframework.boot version '4.0.5'는 존재하지 않는 버전입니다. Spring Boot는 현재 3.x 라인입니다. springBootAdminVersion "4.0.2"도 마찬가지로 확인이 필요합니다. discodeit 모듈과 동일한 버전으로 맞춰주세요.

joonfluence · 2026-04-06T00:33:54Z

@@ -0,0 +1,163 @@
+package com.sprint.mission.discodeit.integraton;
+import static org.springframework.security.test.web.servlet.request.SecurityMockMvcRequestPostProcessors.csrf;


[Minor] 패키지 이름 오타

integraton → integration으로 수정이 필요합니다. 디렉토리명도 함께 변경해주세요.

package com.sprint.mission.discodeit.integration;

joonfluence · 2026-04-06T00:33:55Z

 discodeit.iml
-/storage/
+/storage/
+


[Minor] /storage/ 항목 중복

/storage/가 두 번 정의되어 있습니다. 하나를 제거해주세요.

seonghj added 12 commits March 25, 2026 13:38

dev + prod.yaml 생성

bb00da3

기본 요구사함 프로파일 기반 설정 관리에 맞춘 프로파일 추가

로그 적용 + 커스텀 예외 생성

99cf940

서비스에 커스텀 예외 적용

2e4dd0b

GlobalExceptionHandler 수정

a1e0ea9

build.gradle 수정

8f652c7

Spring Validation 의존성을 추가

유효성 검사 구현

0666e59

주요 Request에 제약 조건 관련 어노테이션 추가, GlobalExceptionHandler에 handleMethodArgumentNotValidException 추가

Spring Boot Actuator 추가

c461643

UserService 단위 테스트

dc3e32f

단위 테스트 BDDMockito 적용

60dfe8a

통합 테스트 추가

cf2ffc2

심화 구현 요소 추가

497032a

controller 로그 추가

5fdcb29

joonfluence reviewed Apr 5, 2026

View reviewed changes

joonfluence reviewed Apr 6, 2026

View reviewed changes

		@@ -0,0 +1,163 @@
		package com.sprint.mission.discodeit.integraton;
		import static org.springframework.security.test.web.servlet.request.SecurityMockMvcRequestPostProcessors.csrf;

		package com.sprint.mission.discodeit.dto.request;


		import jakarta.validation.constraints.*;

Conversation

seonghj commented Apr 5, 2026

요구사항

기본

심화

주요 변경사항

스크린샷

멘토에게

Uh oh!

joonfluence left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

리뷰 요약

Uh oh!

joonfluence left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

joonfluence left a comment •

edited

Loading