[Feat] 티모 9주차 과제 JWT 토큰 방식 회원가입, 로그인 구현 및 마이페이지 개선 - #53
Open
guingguing wants to merge 1 commit into
Open
Conversation
guingguing
changed the title
imminyoung
reviewed
May 27, 2026
| return UserConverter.toMyPageDTO(user); | ||
| } | ||
|
|
||
| public UserResDTO.MyPage getMyPage(AuthUser member) { |
Contributor
There was a problem hiding this comment.
위에 메소드랑 중복 이름인데 같은 역할이면 둘 중 하나 삭제해야할 듯
gyeonseo
reviewed
Jun 2, 2026
gyeonseo
left a comment
gyeonseo
left a comment
There was a problem hiding this comment.
티모몽 JWT 기반 9주차 미션도 다 잘 해줬다! 스웨거 테스트부터 데이타그립 까지 쓰는거보니까 정말 다컸군 ㅎㅎ 데모데이도 여러가지 맡아보면서 성장해봤으면 좋겠다~ 냉동실 문 꼭 닫고다니고.. 9주차 미션 수고많았어 10주차도 파이팅구리야~~
Comment on lines
+13
to
+14
| EMAIL_ALREADY_EXISTS(HttpStatus.BAD_REQUEST, "USER400_1", "이미 사용 중인 이메일입니다."), | ||
| PASSWORD_NOT_MATCH(HttpStatus.UNAUTHORIZED, "USER401_1", "비밀번호가 일치하지 않습니다."); |
There was a problem hiding this comment.
이렇게 비밀번호 가 틀렸다는 걸 그대로 내려주는건 보안상으로는 좋은 느낌은 아냐..!!
이메일 불일치랑 비밀번호 불일치를 다르게 내려주면, 어떤 이메일들이 가입되어있는지 확인할수가 있겠지?? user enumeration 라는 키워드로 찾아보면 이런 관련된 여러 이슈들이 나오니까 한 번 찾아보면 좋겠다!
| return UserConverter.toMyPageDTO(user); | ||
| } | ||
|
|
||
| public UserResDTO.MyPage getMyPage(AuthUser member) { |
There was a problem hiding this comment.
요거 이전에 userId 기반으로 잘 해줬는데 왜 AuthUser 받는 메서드 추가했을까 ?? authUser 는 따지자면 인증 계층에서 쓰이는 객체니까, 서비스 계층까지 넘겨주면 책임이 좀 섞이게 보일 수 있을거같아.
컨트롤러에서 authMember 에서 userId 만 추출해서 넘겨줘서 책임 분리가 된 좋은 설계로 수정해 볼 수 있을듯!!
Comment on lines
+40
to
+43
| .httpBasic(AbstractHttpConfigurer::disable) | ||
| .formLogin(AbstractHttpConfigurer::disable) | ||
| .logout(AbstractHttpConfigurer::disable) | ||
| .sessionManagement(session -> session.sessionCreationPolicy(SessionCreationPolicy.STATELESS)) |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
📂 관련 이슈
🛠️ 작업 사항
📸 관련 이미지 (스크린샷 또는 동영상)
Swagger
DB
💬 기타 설명