[BE] 비밀번호 재설정 기능을 구현한다. #843

tkdgur0906 · 2024-10-21T04:03:38Z

❗ Issue

[BE] 비밀번호 재설정 기능을 구현한다. #807

✨ 구현한 기능

이메일로 인증번호 전송
인증번호 검증
재설정된 비밀번호로 업데이트

📢 논의하고 싶은 내용

🎸 기타

LocalDateTime 테스트를 위해 Clock을 빈으로 등록했습니다.
yml파일 mail관련 설정 업데이트 예저입니다.

shin-jisong

수고하셨습니다~

shin-jisong · 2024-10-21T06:02:30Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/service/AuthService.java

@@ -30,12 +38,16 @@ public class AuthService {

    private static final Logger log = LoggerFactory.getLogger(AuthService.class);
    private static final int GUEST_USER_LIMIT = 1;
+    private static final int PASSWORD_RESET_CODE_EXPIRED_MINUTES = 3;


5분 혹은 10분이 적당하지 않을까요?!

5분으로 변경했습니다!

shin-jisong · 2024-10-21T06:04:03Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/service/AuthService.java

+        }
+
+        userRepository.updatePasswordByEmail(
+                new Email(request.email()), new Password(request.newPassword()));


여기서 패스워드 리셋 코드 저장한 걸 삭제하는 건 어떨까요?!

반영했습니다!

shin-jisong · 2024-10-21T06:05:23Z

backend/bang-ggood/src/main/java/com/bang_ggood/user/repository/UserRepository.java

+    void updatePasswordByEmail(@Param("email") Email email, @Param("newPassword") Password newPassword);
+


로그인 타입도 지정해 줘야 될 것 같아요!

오 완전 예리한 리뷰네요, 수정했습니다!

shin-jisong · 2024-10-21T06:05:58Z

...bang-ggood/src/test/java/com/bang_ggood/auth/repository/PasswordResetCodeRepositoryTest.java

+    @Autowired
+    private PasswordResetCodeRepository passwordResetCodeRepository;
+
+    @DisplayName("특정 시간보다 나중에 생성된 비밀번호 초기화 코드 존재 시 참 반환")


컨벤션이랑 살짝 다른 것 같아서 확인해 주세요!

tsulocalize

간단한 리뷰 남겨드렸습니다~

tsulocalize · 2024-10-21T06:39:25Z

backend/bang-ggood/src/test/java/com/bang_ggood/auth/service/AuthServiceTest.java

+                .hasMessage(ExceptionCode.AUTHENTICATION_PASSWORD_CODE_NOT_FOUND.getMessage());
+    }
+
+    @DisplayName("비밀번호 재설정 선공")


선공 -> 성공

tsulocalize · 2024-10-21T06:39:42Z

backend/bang-ggood/src/test/java/com/bang_ggood/auth/service/MailSenderTest.java

+
+class MailSenderTest {
+
+    @DisplayName("이메일 전송 시 send 메소드 호출")


JINU-CHANG

수고많으셨어요~! 코멘트 몇가지 남겼습니다 🚀

JINU-CHANG · 2024-10-24T13:01:04Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/domain/PasswordResetCode.java

+@Getter
+@NoArgsConstructor(access = PROTECTED)
+@Entity
+public class PasswordResetCode extends BaseEntity {


일회성 데이터 같아서 DB에 저장할 필요는 없다고 생각되는데 혹시 다른 방식 고민해보신게 있나요??

일단 어딘가에 저장되어야하긴 해서 DB에 저장했습니다.
유효시간도 있다보니 레디스에 저장하는게 최선 같은데, 우선은 간단하게 DB에 저장하는 방식으로 구현했습니다!

JINU-CHANG · 2024-10-24T13:02:01Z

...ang-ggood/src/main/java/com/bang_ggood/auth/dto/request/ConfirmPasswordResetCodeRequest.java

+public record ConfirmPasswordResetCodeRequest(String email, String code) {
+}


요기는 Valid 필요없나요??

Dto에서 email 검증하도록 변경했습니다!

Code도 @notblank 필요하지 않나요 👀

code랑 newPassword도 추가해줬습니다!

JINU-CHANG · 2024-10-24T13:05:13Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/service/AuthService.java

@@ -109,6 +121,31 @@ public void logout(String accessToken, String refreshToken, User user) {
        validateTokenOwnership(user, accessAuthUser, refreshAuthUser);
    }

+    public void sendPasswordResetEmail(ForgotPasswordRequest request) {


@transactional 빠진 것 같아요 👀

메서드가 메일 보내는 api랑 같이 묶여있어서 붙이지 않았습니다!
save만 단건으로 트랜잭션 처리되도 괜찮을 것 같아요

JINU-CHANG · 2024-10-24T13:07:52Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/service/AuthService.java

+    private final MailSender mailSender;
+    private final PasswordResetCodeRepository passwordResetCodeRepository;
+    private final Clock clock;


비밀번호 찾기에 대한 코드량이 상당한 것 같은데 새로운 서비스로 분리해보는 건 어떨까요? 해당 클래스들이 의존성을 가질 수 있겠네요!

비밀번호 찾기에 대한 서비스를 만들어서 분리했습니다!

JINU-CHANG · 2024-10-24T13:13:44Z

backend/bang-ggood/src/main/java/com/bang_ggood/global/config/MailConfig.java

+
+    @Value("${spring.mail.properties.mail.smtp.debug}")
+    private boolean debug;
+


공백 👀

JINU-CHANG · 2024-10-24T13:30:54Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/controller/AuthController.java

+        return ResponseEntity.noContent().build();
+    }
+
+    @PostMapping("/v1/password-reset/new-password")


Suggested change

@PostMapping("/v1/password-reset/new-password")

@PostMapping("/v1/password-reset")

마지막은 new-password 단어를 빼도 의미전달이 될 것 같은데 어떠신가용

반영했습니다!

…se-teams/2024-bang-ggood into feat/807-find-password

tsulocalize

코멘트 추가해두었습니다~

tsulocalize · 2024-11-11T05:24:00Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/service/PasswordResetService.java

+    @Transactional(readOnly = true)
+    public void confirmPasswordResetCode(ConfirmPasswordResetCodeRequest request) {
+        LocalDateTime timeLimit = LocalDateTime.now(clock).minusMinutes(PASSWORD_RESET_CODE_EXPIRED_MINUTES);
+        boolean isValid = passwordResetCodeRepository.existsByEmailAndCodeAndCreatedAtAfter(
+                new Email(request.email()), request.code(), timeLimit);
+        if (!isValid) {
+            throw new BangggoodException(ExceptionCode.AUTHENTICATION_PASSWORD_CODE_NOT_FOUND);
+        }
+    }


코드를 확인한 뒤에 확인했다라는 정보를 어디에 기록해두고 있나요?
현재 상태로는 코드 확인 절차 없이도 비밀번호 리셋이 가능한 것 같아 보이네요

코드가 확인되었는지 확인하는 변수를 추가하는 방식으로 변경했습니다!

tsulocalize · 2024-11-11T05:24:36Z

backend/bang-ggood/src/main/java/com/bang_ggood/auth/service/PasswordResetService.java

+        if (!passwordResetCodeRepository.existsByEmailAndCode(email, code)) {
+            throw new BangggoodException(ExceptionCode.AUTHENTICATION_PASSWORD_CODE_NOT_FOUND);
+        }


예시) 한 유저는 리셋 코드를 10개를 미리 발급해두었다. 이후 리셋 코드를 필요할 때마다 기존 발급한 코드를 사용했다.

위 동작을 의도한 건 아니지 않나요?

리셋 코드 생성 시 기존 코드 삭제하도록 수정했습니다!

tsulocalize

수고하셨습니다!

JINU-CHANG

카피 수고많으셨어요 ~~ 👍

shin-jisong

수고하셨습니다!

tkdgur0906 added 11 commits October 20, 2024 19:16

feat: 메일 기본 설정 빈 등록

35a8706

feat: 메일 보내는 MailSender 구현

8f4c7a1

feat: 비밀번호 찾기 시 검증 이메일 보내는 기능 구현

dea07ab

feat: 비밀번호 인증 코드 저장로직 추가

de52e94

feat: 비밀번호 인증 코드 발송 기능 엔드포인트 변경

e9c875c

feat: 비밀번호 인증 코드 저장 시 이메일 같이 저장

8335189

feat: PasswordResetCode 스키마 추가

75e149f

feat: 입력받은 비밀번호 초기화 코드 맞는지 확인하는 기능 구현

583bd8e

feat: 비밀번호 재설정 기능 구현

86eb278

feat: 잘못된 이메일 형식 에러 처리

b0f796c

feat: 에러 메시지 명확히 수정

632e1e5

tkdgur0906 added 🔒 BE 😊 Feature labels Oct 21, 2024

tkdgur0906 added this to the 최종 데모데이 milestone Oct 21, 2024

tkdgur0906 requested review from shin-jisong, tsulocalize and JINU-CHANG October 21, 2024 04:03

tkdgur0906 self-assigned this Oct 21, 2024

test: 비밀번호 초기화 코드 인증 테스트 검증 시간 변경

fdedae3

shin-jisong requested changes Oct 21, 2024

View reviewed changes

tsulocalize reviewed Oct 21, 2024

View reviewed changes

JINU-CHANG requested changes Oct 24, 2024

View reviewed changes

tkdgur0906 changed the base branch from dev to dev-be November 11, 2024 01:41

tkdgur0906 added 7 commits November 11, 2024 10:49

feat: 메일 기본 설정 빈 등록

2071fd2

feat: 메일 보내는 MailSender 구현

7e9f894

feat: 비밀번호 찾기 시 검증 이메일 보내는 기능 구현

b20e715

feat: 비밀번호 인증 코드 저장로직 추가

dd1b87b

feat: 비밀번호 인증 코드 발송 기능 엔드포인트 변경

007507c

feat: 비밀번호 인증 코드 저장 시 이메일 같이 저장

2cb4749

feat: PasswordResetCode 스키마 추가

e8b86a9

tkdgur0906 and others added 17 commits November 11, 2024 10:59

feat: 입력받은 비밀번호 초기화 코드 맞는지 확인하는 기능 구현

6813f8c

feat: 비밀번호 재설정 기능 구현

bd0688e

feat: 잘못된 이메일 형식 에러 처리

87ebeba

feat: 에러 메시지 명확히 수정

58c9b7a

test: 비밀번호 초기화 코드 인증 테스트 검증 시간 변경

464f311

Merge branch 'dev-be' into feat/807-find-password

b30334f

Merge branch 'feat/807-find-password' of https://github.com/woowacour…

2afdc80

…se-teams/2024-bang-ggood into feat/807-find-password

test: 불필요 테스트 코드 삭제

f54dfd1

feat: 인증 코드 유효기간 5분으로 변경

eae3528

feat: 비밀번호 초기화 시 인증 코드 삭제

a105fb6

test: 비밀번호 초기화 코드 유효기간 5분으로 변경 반영

aadbad5

feat: 비밀번호 재설정 시 로그인 타입 Local인 것 반영

f7d9fa9

test: 테스트 displayname 컨벤션 맞게 수정

25dbbbd

refactor: 비밀번호 변경 엔드포인트 수정

b42dc5f

style: 불필요 공백 제거

89c1ac9

feat: dto 이메일 검증 추가

1dc75bf

refactor: 비밀번호 찾기 기능 서비스 분리

e324e31

tsulocalize requested changes Nov 11, 2024

View reviewed changes

tkdgur0906 added 2 commits November 12, 2024 16:26

feat: 비밀번호 초기화 코드가 검증 되었는지 여부 확인하도록 추가

27ddcd7

feat: 비밀번호 초기화 코드 생성 시 기존 이메일에 대한 코드 모두 삭제

673e6da

tsulocalize approved these changes Nov 12, 2024

View reviewed changes

feat: 비밀번호 재설정 dto의 인증코드, 새로운 비밀번호 NotBlank 추가

60b521d

JINU-CHANG approved these changes Nov 13, 2024

View reviewed changes

shin-jisong approved these changes Nov 13, 2024

View reviewed changes

shin-jisong merged commit 79084ee into dev-be Nov 13, 2024
2 checks passed

shin-jisong deleted the feat/807-find-password branch November 13, 2024 07:10

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[BE] 비밀번호 재설정 기능을 구현한다. #843

[BE] 비밀번호 재설정 기능을 구현한다. #843

tkdgur0906 commented Oct 21, 2024

shin-jisong left a comment

shin-jisong Oct 21, 2024

tkdgur0906 Nov 11, 2024

shin-jisong Oct 21, 2024

tkdgur0906 Nov 11, 2024

shin-jisong Oct 21, 2024

tkdgur0906 Nov 11, 2024

shin-jisong Oct 21, 2024

tsulocalize left a comment

tsulocalize Oct 21, 2024

tsulocalize Oct 21, 2024

JINU-CHANG left a comment

JINU-CHANG Oct 24, 2024

tkdgur0906 Nov 11, 2024

JINU-CHANG Oct 24, 2024

tkdgur0906 Nov 11, 2024

JINU-CHANG Nov 12, 2024

tkdgur0906 Nov 13, 2024

JINU-CHANG Oct 24, 2024

tkdgur0906 Nov 11, 2024

JINU-CHANG Oct 24, 2024

tkdgur0906 Nov 11, 2024

JINU-CHANG Oct 24, 2024

JINU-CHANG Oct 24, 2024

tkdgur0906 Nov 11, 2024

tsulocalize left a comment

tsulocalize Nov 11, 2024

tkdgur0906 Nov 12, 2024

tsulocalize Nov 11, 2024

tkdgur0906 Nov 12, 2024

tsulocalize left a comment

JINU-CHANG left a comment

shin-jisong left a comment

		void updatePasswordByEmail(@Param("email") Email email, @Param("newPassword") Password newPassword);


		class MailSenderTest {

		@DisplayName("이메일 전송 시 send 메소드 호출")

		public record ConfirmPasswordResetCodeRequest(String email, String code) {
		}


		@Value("${spring.mail.properties.mail.smtp.debug}")
		private boolean debug;

	@PostMapping("/v1/password-reset/new-password")
	@PostMapping("/v1/password-reset")

[BE] 비밀번호 재설정 기능을 구현한다. #843

[BE] 비밀번호 재설정 기능을 구현한다. #843

Conversation

tkdgur0906 commented Oct 21, 2024

❗ Issue

✨ 구현한 기능

📢 논의하고 싶은 내용

🎸 기타

shin-jisong left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tsulocalize left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

JINU-CHANG left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tsulocalize left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tsulocalize left a comment

Choose a reason for hiding this comment

JINU-CHANG left a comment

Choose a reason for hiding this comment

shin-jisong left a comment

Choose a reason for hiding this comment