[URECA-49] Feat: DB 수정 반영

[joonhyong]

Key Changes

작업 내역

• close: [URECA-49] Feat: DB 변경으로 인한 코드 수정 #25

💬 공유사항 to 리뷰어

• DDL문 확정으로 users테이블, refresh_tokens 테이블의 컬럼 및 구조가 변경됨에 따라 기존의 로직으로는
• 로그인이 안되는 것을 확인하여 auth 도메인 쪽 코드 수정을 하였습니다.
• 또한 카카로 로그인 시에도 이름(넥네임), 이메일이 저장되도록 수정하였습니다.

비고

Summary by CodeRabbit

릴리스 노트

• 버그 수정

  • Kakao 계정으로 로그인할 때 이메일 동의가 필수입니다.
  • 로그아웃 요청에 인증이 필요합니다.

• 기타

  • 데이터베이스 쿼리 및 처리 개선

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

DB 스키마 변경에 따른 백엔드 인증 도메인 코드 조정입니다. Kakao OAuth 클라이언트에 이메일 동의 검증 로직을 추가했으며, MyBatis 매퍼에서 refresh_token_id와 user_id 컬럼 매핑을 명시적으로 설정했습니다. JWT 필터에서 로그아웃 경로를 제외 목록에서 제거하여 인증 필터를 통과하도록 변경했고, OAuthController에서 불필요한 주석을 제거했습니다.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

---

주요 검토 포인트

✅ Kakao OAuth 검증 강화 (긍정적)

이메일 동의 여부(has_email)를 확인한 후 할당하는 방식은 사용자 동의 기반 데이터 수집의 원칙을 따릅니다. 다만 IllegalArgumentException 발생 시 클라이언트에게 어떤 HTTP 상태 코드로 응답하는지 확인이 필요합니다. 이메일 필수 항목이라면 400 Bad Request로 처리하는 것을 권장합니다.

확인 사항:

• 예외 처리 계층에서 IllegalArgumentException을 적절한 HTTP 상태 코드로 변환하는지 검증
• 공식 문서: OAuth 2.0 에러 응답 명세

⚠️ JWT 필터에서 로그아웃 경로 제외 제거

로그아웃 요청이 이제 JWT 검증을 거칩니다. 이는 의도된 보안 강화인지 확인이 필요합니다:

• 로그아웃 시 유효한 토큰 검증이 필수라면 현재 방식은 적절합니다.
• 단, 토큰 만료 후 로그아웃 시도 시 실패할 수 있으므로, 토큰 검증 실패를 우아하게 처리해야 합니다.

개선 제안:
로그아웃 엔드포인트의 에러 처리 정책을 명문화하고, 필터에서 토큰 누락/만료에 대한 명확한 응답을 정의하세요.

🔧 MyBatis 매퍼 설정 변경

refresh_token_id와 user_id를 명시적으로 매핑하고 useGeneratedKeys="true"를 설정한 점은 자동 생성 키를 Java 객체에 올바르게 바인딩합니다.

확인 사항:

• RefreshToken과 User 엔티티의 id 필드가 Long/Integer 타입인지 확인
• 데이터베이스에서 실제로 AUTO_INCREMENT 정책이 적용되어 있는지 검증
• 공식 문서: MyBatis useGeneratedKeys 설정

💡 소프트 딜리트 필터 추가

UserMapper의 deleted_at IS NULL 조건 추가는 논리적 삭제를 지원합니다. 현재 변경에 포함되지 않은 다른 쿼리(update, delete)도 일관된 정책을 따르는지 검토해주세요.

개선 제안:

• 모든 SELECT 쿼리에 동일한 소프트 딜리트 필터를 적용하는 AOP 또는 인터셉터 고려
• 예시: MyBatis Plugin을 활용한 자동 필터링

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	제목이 PR의 주요 변경사항인 DB 스키마 변경 반영을 명확하게 설명하고 있습니다.
Linked Issues check	✅ Passed	DB 테이블 구조 변경(users, refresh_tokens)에 맞춘 auth 도메인 코드 수정이 적용되었고, 카카오 로그인 이메일/닉네임 저장 로직이 추가되었습니다.
Out of Scope Changes check	✅ Passed	JWT 필터에서 logout 경로 제거는 보안/인증 로직과 직접 연관된 변경으로, 전체 인증 흐름 일관성을 위한 범위 내 변경입니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

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.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

src/main/java/com/ureca/unity/domain/auth/service/oauth/KakaoOAuthClient.java (1)

103-119: 이메일 유효성 검증 강화 필요

현재 코드는 has_email만 확인 후 이메일을 사용하는데, 카카오 공식 문서에 따르면 이것만으로는 부족합니다. 다음 필드들도 함께 검증해야 합니다:

• is_email_valid: true여야 카카오 서버에서 실제 이메일이 반환됩니다 (필수)
• is_email_verified: 사용자가 이메일을 카카오에서 검증했는지 확인
• email_needs_agreement: 앱이 이메일 동의(스코프)를 얻어야 하는지 여부

현재 구현은 has_email=true이더라도 이 필드들이 거짓이거나 사용자가 동의하지 않았다면 이메일이 null로 올 수 있는데, 이를 필터링하지 않습니다. is_email_valid를 반드시 체크하고, 필요시 email_needs_agreement를 통해 동의 미흡 여부를 클라이언트에 전달하세요.