Skip to content

[RELEASE] v2.1.8 로그인 통합 오류 해결 #342

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
yongjun0511 merged 2 commits into from
Feb 3, 2026
Merged

[RELEASE] v2.1.8 로그인 통합 오류 해결 #342

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3c57a1c
fix: 기존 유저 정보로 credential 생성
yongjun0511 Feb 3, 2026
40e34e3
Merge pull request #341 from EAT-SSU/fix/login-problem-fix
yongjun0511 Feb 3, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/main/java/ssu/eatssu/domain/auth/service/OAuthService.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ public Tokens kakaoLogin(KakaoLoginRequest request) {
.orElseGet(() -> userRepository.findFirstByEmailOrderByIdAsc(request.email())
.orElseGet(() -> userService.join(request.email(), KAKAO, request.providerId())));

return generateOauthJwtTokens(user.getEmail(), KAKAO, request.providerId());
return generateOauthJwtTokens(user.getEmail(), user.getProvider(), user.getProviderId());
Copy link

@gemini-code-assist gemini-code-assist bot Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

이 메서드 호출은 올바르게 수정되었습니다. 향후 유지보수성을 높이기 위해, generateOauthJwtTokens 메서드가 User 객체를 직접 파라미터로 받도록 리팩토링하는 것을 고려해볼 수 있습니다. 예를 들어, generateOauthJwtTokens(user)와 같이 호출하면 여러 파라미터를 개별적으로 전달할 필요가 없어 코드가 더 간결해지고 명확해집니다. 이 변경은 이번 PR의 범위를 벗어날 수 있으므로 다음 기회에 적용해보시는 것을 추천합니다.

}

/**
Expand All @@ -51,7 +51,7 @@ public Tokens kakaoLoginV2(KakaoLoginRequestV2 request) {
user.updateDeviceType(request.deviceType());


return generateOauthJwtTokens(user.getEmail(), KAKAO, request.providerId());
return generateOauthJwtTokens(user.getEmail(), user.getProvider(), user.getProviderId());
Copy link

@gemini-code-assist gemini-code-assist bot Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

이 메서드 호출은 올바르게 수정되었습니다. 향후 유지보수성을 높이기 위해, generateOauthJwtTokens 메서드가 User 객체를 직접 파라미터로 받도록 리팩토링하는 것을 고려해볼 수 있습니다. 예를 들어, generateOauthJwtTokens(user)와 같이 호출하면 여러 파라미터를 개별적으로 전달할 필요가 없어 코드가 더 간결해지고 명확해집니다. 이 변경은 이번 PR의 범위를 벗어날 수 있으므로 다음 기회에 적용해보시는 것을 추천합니다.

}


Expand All @@ -64,7 +64,7 @@ public Tokens appleLogin(AppleLoginRequest request) {

updateAppleUserEmail(user, oAuthInfo.email());

return generateOauthJwtTokens(user.getEmail(), APPLE, oAuthInfo.providerId());
return generateOauthJwtTokens(user.getEmail(), user.getProvider(), user.getProviderId());
Copy link

@gemini-code-assist gemini-code-assist bot Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

이 메서드 호출은 올바르게 수정되었습니다. 향후 유지보수성을 높이기 위해, generateOauthJwtTokens 메서드가 User 객체를 직접 파라미터로 받도록 리팩토링하는 것을 고려해볼 수 있습니다. 예를 들어, generateOauthJwtTokens(user)와 같이 호출하면 여러 파라미터를 개별적으로 전달할 필요가 없어 코드가 더 간결해지고 명확해집니다. 이 변경은 이번 PR의 범위를 벗어날 수 있으므로 다음 기회에 적용해보시는 것을 추천합니다.

}

/**
Expand All @@ -81,7 +81,7 @@ public Tokens appleLoginV2(AppleLoginRequestV2 request) {

user.updateDeviceType(request.deviceType());

return generateOauthJwtTokens(user.getEmail(), APPLE, oAuthInfo.providerId());
return generateOauthJwtTokens(user.getEmail(), user.getProvider(), user.getProviderId());
Copy link

@gemini-code-assist gemini-code-assist bot Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

이 메서드 호출은 올바르게 수정되었습니다. 향후 유지보수성을 높이기 위해, generateOauthJwtTokens 메서드가 User 객체를 직접 파라미터로 받도록 리팩토링하는 것을 고려해볼 수 있습니다. 예를 들어, generateOauthJwtTokens(user)와 같이 호출하면 여러 파라미터를 개별적으로 전달할 필요가 없어 코드가 더 간결해지고 명확해집니다. 이 변경은 이번 PR의 범위를 벗어날 수 있으므로 다음 기회에 적용해보시는 것을 추천합니다.

}

public Tokens refreshTokens(Authentication authentication) {
Expand Down