Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- CustomerFixture 의 기능과 목적을 설명하는 주석 명시
- 로그인 성공, 로그인 실패 - 이메일 없음, 패스워드 불일치 추가
- SignInCustomerService.signIn 구현 - 이메일 없음과 패스워드 불일치를 같이 처리하면서 보안 강화
- 구매자 로그인에 필요한 email, password 필드 추가
- 구매자 로그인 시 구매자 검색에 필요한 findByEmail 구현
- 구매자 로그인 실패 시 인증 오류 구현
- CustomerControllerTest.testSignInCustomer: 구매자 로그인 테스트 성공 - CustomerControllerTest.testSignInCustomerFail: 구매자 로그인 테스트 실패
- 구매자 로그인 성공 시 OK 반환 - 구매자 로그인 실패 시 BadRequest 반환
- 로그인 실패 시 ErrorCode 추가
- 구매자 로그인을 위한 SignInCustomerRequest 이메일, 패스워드 필드 추가
…매자_로그인 # Conflicts: # src/main/java/camp/woowak/lab/web/api/CustomerController.java # src/test/java/camp/woowak/lab/web/api/customer/CustomerApiControllerTest.java
- 구매자 로그인 실패 시 예외 추가
- 로그인 실패 시 에러코드 구현
- 이메일이 존재하지 않는 지 비밀번호가 존재하지 않는 지 서버 시점에서 명확히 구분할 수 있도록 수정
- UnauthorizedException(ErroCode, String) 생성자 추가
- 로그인 된 구매자의 UUID 를 전달하도록 수정
- CustomerApiControllerTest.testLoginCustomer: 로그인 성공 시 테스트 - CustomerApiControllerTest.testLoginFail: 로그인 실패 시 테스트
- 로그인 인증 후 session 에 LoginCustomer 저장
- 로그인 실패 시 발생하는 예외 처리
- CustomerApiController.login 반환값 추가
- 기존 400 로 관리하던 Status 를 HttpStatus 를 이용해 관리하도록 수정
- 비밀번호가 일치하지 않을 때 예외 던지도록 수정
- 반환값을 Optional 로 랩핑
june-777
approved these changes
Aug 15, 2024
src/main/java/camp/woowak/lab/customer/exception/CustomerErrorCode.java
Outdated
Show resolved
Hide resolved
src/main/java/camp/woowak/lab/web/api/customer/CustomerExceptionHandler.java
Show resolved
Hide resolved
- ProblemDetail 를 생성하는 코드가 겹쳐서 private 메소드로 추출
kimhyun5u
requested review from
a team,
Hyeon-Uk and
Dr-KoKo
and removed request for
a team
Dr-KoKo
approved these changes
Aug 15, 2024
Hyeon-Uk
approved these changes
Aug 15, 2024
리뷰 감사합니다 :3 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
💡 다음 이슈를 해결했어요.
Issue Link - #7
💡 이슈를 처리하면서 추가된 코드가 있어요.
💡 이런 고민을 했어요.
✅ 셀프 체크리스트