Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#748 <- 해당 이슈에 검증 과정 기록해 놓았습니다.
Description
이번 PR에서는 다음과 같은 변경 사항이 적용되었습니다:
에러 클래스 추가:
NotFoundException 클래스를 새로 생성하여, BadRequestException 대신 사용하도록 수정했습니다.
더 정확한 예외 처리를 위해, 리소스가 존재하지 않을 때 NotFoundException을 사용합니다.
비밀번호 찾기 API 개선:
UserService에서 비밀번호 변경 전에 메일 전송을 먼저 수행하도록 로직 순서를 수정했습니다.
dirty checking을 활용하여 userRepository.save() 호출을 제거했습니다.
findByEmailAndNameAndStudentIdAndPhoneNumber 메서드 호출 시, DTO 필드 값에 trim()을 적용해 불필요한 공백이 제거되도록 했습니다.
Controller 리턴 타입 변경:
findPassword 메서드의 리턴 타입을 UserResponseDto에서 void로 수정했습니다. 이 메서드는 비밀번호 재설정 메일 전송을 담당하기 때문에, 별도의 응답 DTO가 필요하지 않습니다.