001-SignUp

[jangjinhong]

📝 AS-IS (코드 바뀌기 전)

✨ To-Be (바뀐 후)

~Service

~Controller

~Repository

🧐 리뷰어들이 봤으면 하는 점

[ ] 소스 코드 위치 경로✔️ [ ] 파일이 많아 놀라셨을텐데 저는 확장성을 고려하여 구현해보았습니다. 규칙대로 하려고 노력했으나, 혹시 소스 코드 위치 경로에 문제가 있다면 말씀해주시면 감사하겠습니다☺️

✔️ TEST 사항

[x] API TEST

크게 3개로 나누겠습니다.

1. 로그인
2. 로그아웃
3. 로그아웃 후 토큰 재발급 시도

📸 테스트 스크린샷

1. 로그인

   • body 설정 필수입니다
   • Authorization 헤더 값에 무조건 Bearder + " " + {token}으로 입력해주셔야 합니다.
   • 콘솔창에 본인의 카카오 이메일, 이름, 프로필 사진 링크가 출력된다면 정상 작동입니다👍🏻

2. 로그아웃

   • 1번에서 응답으로 얻은 accessToken을 복사하여 value값으로 넣어주세요

3. 토큰 재발급

   • 로그아웃 후, 1번에서의 refreshToken으로 재발급을 요청해주세요
   • 로그아웃 된 사용자의 refreshToken은 만료되어 재발급이 불가합니다

[alswl0912]

controller부분에서 kakaotoken, logout, ouath, token 부분으로 다 나누셨는데 이 부분은 LogIn/Out기능으로만 하시는 게 좋을 듯 합니다. ouath와 token부분은 따로 검증하는 컨트롤러를 두는 게 코드 중복이 없을 거 같습니다. 인증 부분은 service에 놓고, controller에는 단순히 요청 및 전달에 집중하는 게 좋겠습니다.

[jangjinhong]

controller부분에서 kakaotoken, logout, ouath, token 부분으로 다 나누셨는데 이 부분은 LogIn/Out기능으로만 하시는 게 좋을 듯 합니다. ouath와 token부분은 따로 검증하는 컨트롤러를 두는 게 코드 중복이 없을 거 같습니다. 인증 부분은 service에 놓고, controller에는 단순히 요청 및 전달에 집중하는 게 좋겠습니다.

인증을 서비스 처리하고, 여러 컨트롤러를 2개로만 관리하도록 수정하겠습니다. 자세하게 알려주셔서 감사합니다.