[feat] 카카오 소셜 로그인 구현 #60
Conversation
|
""" Walkthrough카카오 소셜 로그인을 처리하는 새로운 REST 컨트롤러와 서비스가 추가되었습니다. 카카오 인증 요청 DTO와 응답 DTO를 사용하며, WebClient를 활용해 카카오 사용자 정보를 받아오고, 신규 사용자는 저장 후 JWT 토큰을 발급합니다. 관련 예외 및 DTO, 엔티티, 에러 케이스가 보완되었습니다. Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant KakaoLoginController
participant KakaoLoginService
participant KakaoAPI
participant UserRepository
participant JwtTokenProvider
Client->>KakaoLoginController: POST /api/v1/users/login/kakao (accessToken)
KakaoLoginController->>KakaoLoginService: kakaoLogin(request)
KakaoLoginService->>KakaoAPI: GET /v2/user/me (accessToken)
KakaoAPI-->>KakaoLoginService: KakaoUserInfoDto
KakaoLoginService->>UserRepository: findByEmailAndLoginType(email, KAKAO)
alt 신규 사용자
KakaoLoginService->>UserRepository: save(new User)
end
KakaoLoginService->>JwtTokenProvider: createAccessToken(user)
KakaoLoginService->>JwtTokenProvider: createRefreshToken(user)
KakaoLoginService-->>KakaoLoginController: KakaoLoginResponseDto
KakaoLoginController-->>Client: CommonResponse<KakaoLoginResponseDto>
Estimated code review effort3 (90–240분) Possibly related issues
Suggested labels
Suggested reviewers
Poem
✨ Finishing Touches
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. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 10
♻️ Duplicate comments (1)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
64-67: 일관된 인증 로직이 적용되었습니다.
getUserPlaces메서드에서도 동일한 인증 방식을 사용하여 일관성을 유지하고 있습니다. 다만 여기서도 동일한 캐스팅 안전성 문제가 있으니 위에서 제안한 방식으로 개선하시기 바랍니다.
🧹 Nitpick comments (4)
src/main/java/com/wayble/server/user/service/UserPlaceService.java (1)
75-75: 이미지 URL 조회 로직이 효율적으로 개선되었습니다.기존의 스트림 방식에서
getMainImageUrl()직접 호출로 변경하여 성능이 개선되었습니다. WaybleZone 엔티티의mainImageUrl필드 추가와 잘 연계된 변경사항입니다.다만
mainImageUrl이 null인 경우에 대한 처리를 고려해보시기 바랍니다:- String imageUrl = waybleZone.getMainImageUrl(); + String imageUrl = waybleZone.getMainImageUrl() != null ? + waybleZone.getMainImageUrl() : "default_image_url";src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
39-39: Authorization 헤더 파라미터가 불필요합니다.현재 코드에서는
Authorization헤더를 파라미터로 받고 있지만 실제로는 사용하지 않습니다. Spring Security에서 자동으로 처리하므로 이 파라미터는 제거하는 것이 좋습니다.public CommonResponse<String> saveUserPlace( @PathVariable Long userId, - @RequestBody @Valid UserPlaceRequestDto request, - @RequestHeader(value = "Authorization") String authorizationHeader + @RequestBody @Valid UserPlaceRequestDto request ) {public CommonResponse<List<UserPlaceListResponseDto>> getUserPlaces( - @PathVariable Long userId, - @RequestHeader("Authorization") String authorizationHeader + @PathVariable Long userId ) {Also applies to: 62-62
src/main/java/com/wayble/server/user/dto/KakaoLoginResponseDto.java (1)
12-17: 중첩 UserDto 문서화 추가중첩된 UserDto에도 적절한 문서화가 필요합니다.
+ /** + * 사용자 기본 정보 DTO + */ @Builder public record UserDto( + /** 사용자 ID */ Long id, + /** 닉네임 */ String nickname, + /** 이메일 */ String email ) {}src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java (1)
30-40: 사용자 정보 추출 로직이 적절합니다.카카오 API 응답에서 필요한 사용자 정보를 올바르게 추출하고 있습니다. 다만,
kakaoId변수는 현재 사용되지 않고 있는 것 같습니다.만약 향후 사용할 계획이 없다면 해당 라인을 제거하거나, 사용할 계획이 있다면 주석으로 설명을 추가해주세요.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (11)
src/main/java/com/wayble/server/user/controller/KakaoLoginController.java(1 hunks)src/main/java/com/wayble/server/user/controller/UserPlaceController.java(3 hunks)src/main/java/com/wayble/server/user/dto/KakaoLoginRequestDto.java(1 hunks)src/main/java/com/wayble/server/user/dto/KakaoLoginResponseDto.java(1 hunks)src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java(1 hunks)src/main/java/com/wayble/server/user/dto/UserPlaceRequestDto.java(0 hunks)src/main/java/com/wayble/server/user/entity/User.java(1 hunks)src/main/java/com/wayble/server/user/exception/UserErrorCase.java(1 hunks)src/main/java/com/wayble/server/user/service/UserPlaceService.java(2 hunks)src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java(1 hunks)src/main/java/com/wayble/server/wayblezone/entity/WaybleZone.java(1 hunks)
💤 Files with no reviewable changes (1)
- src/main/java/com/wayble/server/user/dto/UserPlaceRequestDto.java
🧰 Additional context used
🧠 Learnings (2)
src/main/java/com/wayble/server/wayblezone/entity/WaybleZone.java (1)
Learnt from: seung-in-Yoo
PR: Wayble-Project/wayble-spring#37
File: src/main/java/com/wayble/server/user/entity/User.java:61-62
Timestamp: 2025-07-13T15:08:59.318Z
Learning: Java에서 같은 패키지 내의 클래스들은 import 구문 없이 서로를 참조할 수 있다. com.wayble.server.user.entity 패키지 내의 클래스들은 명시적인 import 없이 사용 가능하다.
src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java (1)
Learnt from: seung-in-Yoo
PR: Wayble-Project/wayble-spring#37
File: src/main/java/com/wayble/server/user/entity/User.java:61-62
Timestamp: 2025-07-13T15:08:59.318Z
Learning: Java에서 같은 패키지 내의 클래스들은 import 구문 없이 서로를 참조할 수 있다. com.wayble.server.user.entity 패키지 내의 클래스들은 명시적인 import 없이 사용 가능하다.
🧬 Code Graph Analysis (2)
src/main/java/com/wayble/server/user/dto/KakaoLoginResponseDto.java (1)
src/main/java/com/wayble/server/common/response/CommonResponse.java (1)
Builder(8-42)
src/main/java/com/wayble/server/user/controller/KakaoLoginController.java (1)
src/main/java/com/wayble/server/common/config/security/jwt/JwtAuthenticationFilter.java (1)
RequiredArgsConstructor(13-43)
🔇 Additional comments (11)
src/main/java/com/wayble/server/wayblezone/entity/WaybleZone.java (1)
61-63: 대표 이미지 필드 추가는 성능 개선에 도움이 됩니다.기존 방식에서 이미지 리스트를 스트림으로 조회하던 것을 직접 필드로 저장하여 성능을 개선한 좋은 변경사항입니다.
다만 이 필드가 nullable로 설정되어 있는데, 대표 이미지가 없는 경우에 대한 기본값 처리나 검증 로직을 고려해보시기 바랍니다.
src/main/java/com/wayble/server/user/service/UserPlaceService.java (3)
32-32: 메서드 시그니처 개선이 보안을 강화합니다.
userId를 별도 파라미터로 받는 것은 컨트롤러에서 인증된 사용자 ID를 직접 전달하여 보안을 강화하는 좋은 변경사항입니다.
34-34: 파라미터 활용 변경이 일관성을 개선합니다.인증된
userId파라미터를 직접 사용하여 사용자 조회와 중복 확인을 수행하는 것이 더 명확하고 안전합니다.Also applies to: 42-42
32-46: 호출부 검토 완료: 메서드 시그니처가 올바르게 업데이트되었습니다.–
UserPlaceController.java(36행): 컨트롤러 메서드 시그니처
–UserPlaceController.java(46행):userPlaceService.saveUserPlace(userId, request)호출위 두 곳 모두 새 시그니처(
Long userId, UserPlaceRequestDto request)에 맞춰 수정되어 있어 호환성에 문제가 없습니다.src/main/java/com/wayble/server/user/controller/UserPlaceController.java (2)
46-46: 서비스 메서드 호출이 올바르게 업데이트되었습니다.
userId파라미터를 별도로 전달하는 방식으로 변경된 것이 서비스 레이어의 변경사항과 일치합니다.
33-35: API 응답 문서가 적절히 업데이트되었습니다.403 Forbidden 응답 케이스를 추가한 것이 변경된 인증 로직과 일치합니다.
src/main/java/com/wayble/server/user/exception/UserErrorCase.java (1)
17-18: 카카오 인증 실패 에러 케이스 추가 적절함새로운
KAKAO_AUTH_FAILED에러 케이스가 적절하게 추가되었습니다. HTTP 상태 코드 401과 에러 메시지가 카카오 인증 실패 상황에 적합합니다.src/main/java/com/wayble/server/user/controller/KakaoLoginController.java (3)
17-19: 컨트롤러 구조가 잘 설계되었습니다.Spring Boot의 표준 패턴을 잘 따르고 있으며, 의존성 주입과 요청 매핑이 적절히 구현되어 있습니다.
24-30: API 문서화가 잘 되어 있습니다.OpenAPI 어노테이션을 통해 엔드포인트에 대한 명확한 설명과 응답 코드가 문서화되어 있어 좋습니다.
31-36: 전역 예외 처리기 확인 완료
src/main/java/com/wayble/server/common/exception/GlobalExceptionHandler.java에서
@ExceptionHandler(ApplicationException.class)를 통해ApplicationException을 처리하고 있으므로,
컨트롤러 단에서 별도 예외 처리 로직을 추가할 필요가 없습니다.src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java (1)
65-80: JWT “role” 클레임 처리 로직 확인 요청
- JwtTokenProvider.generateToken(Long, String)에서
userType.name()을"role"클레임으로 설정하고 있습니다. UserType enum 값(DISABLED,COMPANION)과 일치합니다.- JwtTokenProvider에는 토큰 유효성 검사(validateToken), 사용자 ID 파싱(getUserId), 만료시간 파싱(getTokenExpiry) 메서드만 구현되어 있습니다.
• “role” 클레임을 파싱해서 GrantedAuthority로 변환하고 Authentication 객체에 반영하는 로직이 있는지 확인해주세요.
• 보통 JwtAuthenticationFilter 또는 SecurityConfig 내 getAuthentication 메서드에서 처리됩니다.- 인증/인가 전체 흐름에서 “role” 클레임이 올바르게 읽히고 권한 부여에 사용되는지 검증 부탁드립니다.
| Long tokenUserId = (Long) SecurityContextHolder.getContext().getAuthentication().getPrincipal(); | ||
| if (!userId.equals(tokenUserId)) { | ||
| throw new ApplicationException(UserErrorCase.FORBIDDEN); | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
Spring Security 컨텍스트 사용이 좋은 개선입니다.
JWT 토큰을 수동으로 파싱하던 방식에서 Spring Security의 인증 컨텍스트를 사용하는 것으로 변경한 것은 표준적이고 안전한 접근법입니다.
다만 타입 캐스팅에서 ClassCastException이 발생할 수 있습니다.
안전한 캐스팅을 위해 다음과 같이 개선하는 것을 제안합니다:
- Long tokenUserId = (Long) SecurityContextHolder.getContext().getAuthentication().getPrincipal();
+ Authentication authentication = SecurityContextHolder.getContext().getAuthentication();
+ if (!(authentication.getPrincipal() instanceof Long)) {
+ throw new ApplicationException(UserErrorCase.FORBIDDEN);
+ }
+ Long tokenUserId = (Long) authentication.getPrincipal();📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| Long tokenUserId = (Long) SecurityContextHolder.getContext().getAuthentication().getPrincipal(); | |
| if (!userId.equals(tokenUserId)) { | |
| throw new ApplicationException(UserErrorCase.FORBIDDEN); | |
| } | |
| Authentication authentication = SecurityContextHolder.getContext().getAuthentication(); | |
| if (!(authentication.getPrincipal() instanceof Long)) { | |
| throw new ApplicationException(UserErrorCase.FORBIDDEN); | |
| } | |
| Long tokenUserId = (Long) authentication.getPrincipal(); | |
| if (!userId.equals(tokenUserId)) { | |
| throw new ApplicationException(UserErrorCase.FORBIDDEN); | |
| } |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/user/controller/UserPlaceController.java
around lines 41 to 44, the code casts the principal object from the Spring
Security context directly to Long, which can cause ClassCastException. To fix
this, retrieve the principal as an Object, check its type safely, and extract
the user ID accordingly, for example by checking if the principal is an instance
of a UserDetails implementation or a String, then convert or parse it to Long
before comparing with userId.
| public record KakaoLoginRequestDto( | ||
| String accessToken | ||
| ) {} No newline at end of file |
There was a problem hiding this comment.
🛠️ Refactor suggestion
입력 검증 및 문서화 추가 필요
카카오 액세스 토큰에 대한 유효성 검사가 없어 보안상 위험할 수 있습니다.
다음과 같이 개선하세요:
package com.wayble.server.user.dto;
+import jakarta.validation.constraints.NotBlank;
+
+/**
+ * 카카오 로그인 요청 DTO
+ */
public record KakaoLoginRequestDto(
+ @NotBlank(message = "액세스 토큰은 필수입니다")
String accessToken
) {}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| public record KakaoLoginRequestDto( | |
| String accessToken | |
| ) {} | |
| package com.wayble.server.user.dto; | |
| import jakarta.validation.constraints.NotBlank; | |
| /** | |
| * 카카오 로그인 요청 DTO | |
| */ | |
| public record KakaoLoginRequestDto( | |
| @NotBlank(message = "액세스 토큰은 필수입니다") | |
| String accessToken | |
| ) {} |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/user/dto/KakaoLoginRequestDto.java around
lines 3 to 5, the KakaoLoginRequestDto record lacks validation for the
accessToken field, which poses a security risk. Add input validation annotations
such as @NotBlank or @NotNull to ensure the accessToken is provided and not
empty. Additionally, include JavaDoc comments to document the purpose and
constraints of the accessToken field for better clarity and maintainability.
| @Builder | ||
| public record KakaoLoginResponseDto( | ||
| String accessToken, | ||
| String refreshToken, | ||
| boolean isNewUser, | ||
| UserDto user | ||
| ) { |
There was a problem hiding this comment.
🛠️ Refactor suggestion
문서화 및 보안 고려사항 추가 필요
응답 DTO에 문서화가 부족하고 민감한 토큰 정보에 대한 보안 고려사항이 필요합니다.
package com.wayble.server.user.dto;
import lombok.Builder;
+/**
+ * 카카오 로그인 응답 DTO
+ */
@Builder
public record KakaoLoginResponseDto(
+ /** JWT 액세스 토큰 */
String accessToken,
+ /** JWT 리프레시 토큰 */
String refreshToken,
+ /** 신규 사용자 여부 */
boolean isNewUser,
+ /** 사용자 정보 */
UserDto user
) {📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| @Builder | |
| public record KakaoLoginResponseDto( | |
| String accessToken, | |
| String refreshToken, | |
| boolean isNewUser, | |
| UserDto user | |
| ) { | |
| package com.wayble.server.user.dto; | |
| import lombok.Builder; | |
| /** | |
| * 카카오 로그인 응답 DTO | |
| */ | |
| @Builder | |
| public record KakaoLoginResponseDto( | |
| /** JWT 액세스 토큰 */ | |
| String accessToken, | |
| /** JWT 리프레시 토큰 */ | |
| String refreshToken, | |
| /** 신규 사용자 여부 */ | |
| boolean isNewUser, | |
| /** 사용자 정보 */ | |
| UserDto user | |
| ) { | |
| } |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/user/dto/KakaoLoginResponseDto.java around
lines 5 to 11, the response DTO lacks proper documentation and does not address
security concerns for sensitive token fields. Add JavaDoc comments to the class
and its fields explaining their purpose. Additionally, consider marking the
accessToken and refreshToken fields to be excluded from logs or serialization
where appropriate to enhance security.
| @Getter | ||
| @Setter | ||
| public class KakaoUserInfoDto { | ||
| private Long id; | ||
| private KakaoAccount kakao_account; |
There was a problem hiding this comment.
🛠️ Refactor suggestion
불변성과 JSON 매핑 개선 필요
DTO에서 @Setter 사용은 객체의 불변성을 해치고 예상치 못한 변경을 허용할 수 있습니다.
Jackson 어노테이션을 사용하여 불변 DTO로 개선하세요:
package com.wayble.server.user.dto;
+import com.fasterxml.jackson.annotation.JsonProperty;
import lombok.Getter;
-import lombok.Setter;
@Getter
-@Setter
public class KakaoUserInfoDto {
private Long id;
+ @JsonProperty("kakao_account")
- private KakaoAccount kakao_account;
+ private KakaoAccount kakaoAccount;📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| @Getter | |
| @Setter | |
| public class KakaoUserInfoDto { | |
| private Long id; | |
| private KakaoAccount kakao_account; | |
| package com.wayble.server.user.dto; | |
| import com.fasterxml.jackson.annotation.JsonProperty; | |
| import lombok.Getter; | |
| @Getter | |
| public class KakaoUserInfoDto { | |
| private Long id; | |
| @JsonProperty("kakao_account") | |
| private KakaoAccount kakaoAccount; | |
| // … any other fields or nested classes … | |
| } |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java around lines 6
to 10, the use of @Setter allows modification of the DTO fields, which breaks
immutability and can lead to unexpected changes. Remove the @Setter annotation
and make the fields final. Add a constructor annotated with @JsonCreator and use
@JsonProperty on the parameters to enable proper JSON mapping while keeping the
DTO immutable.
| @Getter | ||
| @Setter | ||
| public static class KakaoAccount { | ||
| private String email; | ||
| private Profile profile; | ||
|
|
||
| @Getter | ||
| @Setter | ||
| public static class Profile { | ||
| private String nickname; | ||
| private String profile_image_url; | ||
| } | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
중첩 클래스들도 불변성 적용 필요
모든 중첩 클래스에서도 @Setter를 제거하고 Jackson 어노테이션을 추가해야 합니다.
@Getter
- @Setter
public static class KakaoAccount {
private String email;
private Profile profile;
@Getter
- @Setter
public static class Profile {
private String nickname;
+ @JsonProperty("profile_image_url")
- private String profile_image_url;
+ private String profileImageUrl;
}
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| @Getter | |
| @Setter | |
| public static class KakaoAccount { | |
| private String email; | |
| private Profile profile; | |
| @Getter | |
| @Setter | |
| public static class Profile { | |
| private String nickname; | |
| private String profile_image_url; | |
| } | |
| } | |
| @Getter | |
| public static class KakaoAccount { | |
| private String email; | |
| private Profile profile; | |
| @Getter | |
| public static class Profile { | |
| private String nickname; | |
| @JsonProperty("profile_image_url") | |
| private String profileImageUrl; | |
| } | |
| } |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/user/dto/KakaoUserInfoDto.java from lines 12
to 24, remove all @Setter annotations from the nested classes to enforce
immutability. Add appropriate Jackson annotations such as @JsonProperty to map
JSON fields to Java fields correctly without setters. This will make the nested
classes immutable and compatible with Jackson deserialization.
src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java
Show resolved
Hide resolved
src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java
Show resolved
Hide resolved
src/main/java/com/wayble/server/user/service/auth/KakaoLoginService.java
Show resolved
Hide resolved
| // 혹시 필요할수도 있어서 추가해놓음 | ||
| public void setMainImageUrl(String mainImageUrl) { | ||
| this.mainImageUrl = mainImageUrl; | ||
| } |
There was a problem hiding this comment.
🛠️ Refactor suggestion
엔티티 설계 일관성을 고려해보세요.
현재 엔티티는 Lombok의 @Getter만 사용하고 있는데, 수동으로 setter를 추가하는 것은 일관성에 맞지 않습니다.
또한 주석의 톤이 비공식적입니다.
다음과 같이 개선을 제안합니다:
- // 혹시 필요할수도 있어서 추가해놓음
- public void setMainImageUrl(String mainImageUrl) {
- this.mainImageUrl = mainImageUrl;
- }
+ /**
+ * 대표 이미지 URL을 설정합니다.
+ * @param mainImageUrl 설정할 대표 이미지 URL
+ */
+ public void updateMainImageUrl(String mainImageUrl) {
+ this.mainImageUrl = mainImageUrl;
+ }또는 엔티티의 불변성을 유지하고 생성 시에만 설정하도록 @Builder의 @Builder.Default 사용을 고려해보세요.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| // 혹시 필요할수도 있어서 추가해놓음 | |
| public void setMainImageUrl(String mainImageUrl) { | |
| this.mainImageUrl = mainImageUrl; | |
| } | |
| /** | |
| * 대표 이미지 URL을 설정합니다. | |
| * @param mainImageUrl 설정할 대표 이미지 URL | |
| */ | |
| public void updateMainImageUrl(String mainImageUrl) { | |
| this.mainImageUrl = mainImageUrl; | |
| } |
🤖 Prompt for AI Agents
In src/main/java/com/wayble/server/wayblezone/entity/WaybleZone.java around
lines 65 to 68, the setter method for mainImageUrl is manually added while the
class uses Lombok's @Getter only, causing inconsistency. Remove the manual
setter and instead use Lombok annotations like @Setter if setters are needed, or
better, maintain immutability by setting the field via constructor or @Builder
with @Builder.Default. Also, replace the informal comment with a clear,
professional explanation or remove it if unnecessary.
iOS/안드로이드 앱에서는 리다이렉트 URL 자체로 보여주지 않고 보통 앱에서 카카오 SDK를 사용해서 앱이 인가코드 또는 access token을 직접 받는다고 하더라구요! 그래서 서버는 프론트가 받은 access token만 전달받아서 처리하면 되는거 같아요 |
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
39-42: SecurityContext에서 Long으로 직접 캐스팅 시 ClassCastException 위험
SecurityContextHolder에서 꺼낸principal을 바로Long으로 캐스팅하고 있습니다.
principal타입이 언제든지Long이 아닐 수 있어 런타임에ClassCastException이 발생할 수 있습니다. 이전 리뷰에서도 동일 지적이 있었으므로, 안전한 타입 체크 로직으로 교체해 주세요.Also applies to: 61-64
🧹 Nitpick comments (1)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java (1)
35-44:@AuthenticationPrincipal활용으로 코드 단순화 가능메서드 시그니처에
@AuthenticationPrincipal Long tokenUserId를 추가하면 SecurityContext에서 직접 꺼내고 캐스팅할 필요가 없어집니다. 예시:public CommonResponse<String> saveUserPlace( @PathVariable Long userId, @RequestBody @Valid UserPlaceRequestDto request, @AuthenticationPrincipal Long tokenUserId ) { if (!userId.equals(tokenUserId)) { throw new ApplicationException(UserErrorCase.FORBIDDEN); } ... }가독성과 안전성이 동시에 개선되니 검토해 보시기 바랍니다.
Also applies to: 58-66
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/main/java/com/wayble/server/user/controller/UserPlaceController.java(2 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: run-docker-image-on-ec2
|
오호 그렇게 처리하는군요 하나 배워갑니다...! 수고 많으셨어요!! |
#️⃣ 연관된 이슈
#58
📝 작업 내용
카카오 소셜 로그인 API 구현
🖼️ 스크린샷 (선택)
client_id,redirect_uri 설정 후 들어와지는 화면
예시 url: https://kauth.kakao.com/oauth/authorize?client_id=REST_API_KEY&redirect_uri=http://localhost:8080/api/v1/users/login/kakao&response_type=code&scope=profile_nickname,account_email
카카오 디벨로퍼 -> 동의항목
카카오에서 지급한 토큰 발급 성공 (postman)
지급받은 토큰으로 직접 swagger에서 테스트 진행 후 로그인 성공 (swagger)
💬 리뷰 요구사항 (선택)
Summary by CodeRabbit
신규 기능
버그 수정
기타