[FEAT] 9주차 미션_링크 - #41
Conversation
wnsgh7368
left a comment
There was a problem hiding this comment.
전체적으로 다 비슷해서 이번주도 리뷰 남길게 많이 없는 듯! 이번주도 고생 많았어!
There was a problem hiding this comment.
/home 이랑 /mypage API가 둘 다 컨트롤러에서 memberService의 getmyPage()를 호출하고 있어! 응답도 마찬가지로 둘 다 HomeResponseDTO에 담고 있는데 둘이 섞였나?? 확인 한번 해봐야할 것 같아!
There was a problem hiding this comment.
@Email 어노테이션으로 검증해주는거 좋네! 하지만 @Email 어노테이션은 허점이 많아서 검증이 제대로 안될 수 있다고 하네! 첨부한 링크 가볍게 읽어보는거 추천해~(링크가 아래 링크 읽는거 추천)
https://velog.io/@mj3242/Spring%EC%97%90%EC%84%9C-%EC%9D%B4%EB%A9%94%EC%9D%BC%EC%9D%84-%EA%B2%80%EC%A6%9D%ED%95%98%EB%8A%94-%EB%B0%A9%EB%B2%95
gyeonseo
left a comment
There was a problem hiding this comment.
링크 JWT 기반 회원가입 로그인 다 무난하게 잘 구현해줬네~~ 링크도 항상 미션 잘해와주고 9주차까지 미션 안밀리고 잘해줘서 고맙다 ㅎㅎ 데모데이도 일정 조율 잘 될거야!! 너무 걱정하지말고 좋은 팀 만나서 여름동안 재밌게 개발해보면 좋겠다 9주차 미션 수고많았어 데모데이도 화이팅이야!!
| public class UserLoginResponseDTO { | ||
|
|
||
| // JWT Access Token입니다. | ||
| private String accessToken; |
There was a problem hiding this comment.
로그인 응답으로 요렇게 액세스토큰 자체만 내려주는구나! 나는 만료시간까지 같이 내려주는걸 추천하긴 해! 만료시간이 왜 필요한지도 링크가 한번 찾아보면 좋겠다..!
|
|
||
| MEMBER_NOT_FOUND(HttpStatus.NOT_FOUND, "MEMBER404", "사용자를 찾을 수 없습니다."), | ||
| MEMBER_PASSWORD_NOT_MATCH(HttpStatus.UNAUTHORIZED, "MEMBER401", "비밀번호가 일치하지 않습니다."), | ||
| MEMBER_EMAIL_ALREADY_EXISTS(HttpStatus.CONFLICT, "MEMBER409", "이미 가입된 이메일입니다."); |
There was a problem hiding this comment.
이렇게 분기하면 로그인 API에서 이메일 가입 여부를 추측할 수 있게되니까, 로그인 실패는 하나로 합치는게 더 좋아보인다!!
📂 관련 이슈
🛠️ 작업 사항
JWT 기반 인증 구현
로그인 API 구현
회원가입 API 보완
마이페이지 API 개선
Security 설정 변경
📸 관련 이미지 (스크린샷 또는 동영상)
💬 기타 설명