feat(be):소셜 로그인 구현

[jungdongha]

깃허브 잘못만진거같아서 다시 올립니다

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/common/security/CustomOAuth2UserService.kt

🟢 좋은점:

• Kotlin의 when 표현식을 활용해 provider별 파싱 로직을 명확하게 분기 처리하여 가독성과 유지보수성을 높임. 이는 Kotlin 최적화 규칙(when 표현식 사용)을 잘 따름.
• OAuthUserInfo data class를 반환 타입으로 사용하며, null safety를 고려한 캐스팅(as?, as String?)을 통해 안전한 데이터 추출을 구현함. 특히 Kakao 파싱에서 중첩된 Map 구조를 안전하게 처리한 점이 우수함.
• ktlint 규칙 준수: 네이밍 컨벤션(예: parseNaver, parseKakao 함수명)이 일관되며, 코드 포맷팅(들여쓰기, 공백)이 깔끔함. 주석으로 Kakao 이메일 처리 이유를 설명해 문서화가 적절함.

🟡 개선사항:

• Kakao email 처리에서 "kakao_$[email protected]" 같은 하드코딩 대신, 상수나 설정 파일(예: application.yml)에서 기본 도메인을 관리하면 유연성이 높아질 수 있음. 또한, OAuthUserInfo에 email 필드가 optional(nullable)로 설계되어 있다면 강제 생성 대신 null 허용을 고려.
• parseNaver와 parseKakao 함수에서 Map 캐스팅 실패 시(예: 예상 키가 없거나 타입 불일치) ClassCastException이 발생할 수 있음. Elvis 연산자(?:)나 let/with 등의 Kotlin idiomatic 패턴을 더 활용해 기본값 처리(예: nickname = response["name"] as? String ?: "default_user")를 강화하면 null safety가 더 강화됨.
• 확장 함수나 유틸리티 함수(예: fun Map<String, Any>.safeGetString(key: String): String?)를 도입하면 중복 캐스팅 코드를 줄이고 Kotlin 최적화 측면에서 더 나아질 수 있음.

🔴 문제점:

• parseKakao에서 attributes["id"].toString()으로 oauthId를 처리하는데, Kakao의 "id"는 보통 Long 타입이므로 toString()이 안전하지만, 만약 null이거나 예상치 못한 타입이면 NPE나 예외가 발생할 수 있음. 반드시 attributes["

[github-actions]

🤖 AI 리뷰 - src/main/kotlin/com/back/koreaTravelGuide/common/security/SecurityConfig.kt

🟢 좋은점: Kotlin 최적화 측면에서 우수합니다. addFilterBefore 메서드 호출 시 제네릭 타입 <UsernamePasswordAuthenticationFilter>를 명시적으로 사용함으로써 타입 안전성을 강화하고, Java 스타일의 ::class.java 리플렉션을 피한 Kotlin다운 표현으로 코드가 더 간결하고 효율적으로 개선되었습니다. 이는 Kotlin의 제네릭 지원을 잘 활용한 예입니다.

🟡 개선사항: 없음. 변경이 작고 목적에 부합합니다.

🔴 문제점: 없음. 글로벌 익셉션 처리나 ApiResponse 규칙은 이 Security 설정 파일과 직접 관련이 없어 적용되지 않으며, ktlint 규칙(포맷팅, 네이밍)도 준수됩니다.