#91 🐛 fix: 회원가입시 토큰 반환

[kkshyun]

📍 PR 타입 (하나 이상 선택)

• 기능 추가
• 버그 수정
• 의존성, 환경 변수, 빌드 관련 코드 업데이트
• 기타 사소한 수정

❗️ 관련 이슈 링크

Close #

📌 개요

• 회원가입시 토큰 반환

🔁 변경 사항

회원가입시 토큰 반환

📸 스크린샷

👀 기타 더 이야기해볼 점

✅ 체크 리스트

• PR 템플릿에 맞추어 작성했어요.
• 변경 내용에 대한 테스트를 진행했어요.
• 프로그램이 정상적으로 동작해요.
• PR에 적절한 라벨을 선택했어요.
• 불필요한 코드는 삭제했어요.

Summary by CodeRabbit

• 새로운 기능
  • 회원가입 후 자동 로그인: 회원가입을 완료하면 인증 토큰이 즉시 발급되어 별도의 로그인 과정 없이 바로 서비스를 이용할 수 있게 개선되었습니다.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

회원가입 API의 반환 타입이 DataResponse<Void>에서 DataResponse<TokenResponse>로 변경되었습니다. 회원가입 후 자동으로 사용자를 로그인 시켜 토큰을 반환하는 기능이 추가되었습니다.

Changes

Cohort / File(s)	Summary
AuthController 회원가입 엔드포인트 src/main/java/com/finsight/finsight/domain/auth/presentation/AuthController.java	회원가입 메서드의 반환 타입을 변경하고, 회원 생성 후 자동 로그인하여 TokenResponse를 반환하도록 로직 수정

🎯 검토 포인트 (시니어 관점)

좋은 점:

• 사용자 경험 개선: 회원가입 후 별도의 로그인 요청 없이 토큰을 바로 받을 수 있습니다. 좋은 개선입니다! ✨

확인이 필요한 부분:

1. 에러 처리: 회원가입 성공 후 로그인 과정에서 실패하면 어떻게 처리됩니까? 이미 생성된 계정이 남아있을 텐데, 이 부분의 트랜잭션 처리가 명확해야 합니다.
2. 보안: 회원가입 요청 직후 즉시 로그인하는 것이 의도된 디자인인지 확인이 필요합니다. 이메일 인증 프로세스가 있다면 순서를 다시 검토해야 할 수 있습니다.
3. 의존성: 로그인 로직(메서드)이 정상 작동하는지 테스트 커버리지 확인 부탁드립니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch docs/#91-documentation

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}