Skip to content

Commit

Permalink
Merge pull request #4 from 28th-meetup/fix/common
Browse files Browse the repository at this point in the history
fix: 카카오 로그인 오류 수정
  • Loading branch information
eckrin committed Oct 18, 2023
2 parents 0aa0915 + 6af378b commit 5c80fc4
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions src/main/java/com/kusitms/jipbap/auth/AuthService.java
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
import com.kusitms.jipbap.user.Role;
import com.kusitms.jipbap.user.User;
import com.kusitms.jipbap.user.UserRepository;
import com.kusitms.jipbap.user.exception.UserNotFoundException;
import lombok.RequiredArgsConstructor;
import lombok.extern.slf4j.Slf4j;
import org.springframework.beans.factory.annotation.Value;
Expand All @@ -28,6 +29,7 @@
import org.springframework.web.client.RestTemplate;

import java.lang.reflect.Member;
import java.util.Optional;

@Slf4j
@Service
Expand Down Expand Up @@ -142,9 +144,7 @@ public SignInResponseDto kakaoAutoSignIn(KakaoProfileDto profile) {
.oauth(KAKAO)
.build();

User findUser = userRepository.findByEmail(kakaoUser.getEmail()).orElse(null);

if(findUser == null) {
if(userRepository.findByEmail(kakaoUser.getEmail()).isEmpty()) {
log.info(profile.getKakao_account().getEmail()+": 기존 회원이 아니므로 자동 회원가입 후 로그인을 진행합니다.");
signUp(new SignUpRequestDto(
kakaoUser.getEmail(),
Expand All @@ -155,10 +155,12 @@ public SignInResponseDto kakaoAutoSignIn(KakaoProfileDto profile) {
Role.USER,
kakaoUser.getImage()
));
findUser.updateOAuth(KAKAO);
} else {
log.info(profile.getKakao_account().getEmail()+": 기존 회원이므로 자동 로그인을 진행합니다.");
}

User findUser = userRepository.findByEmail(kakaoUser.getEmail()).orElseThrow(()->new UserNotFoundException("카카오 회원가입 도중 문제가 발생했습니다."));
findUser.updateOAuth(KAKAO);
return signIn(kakaoUser.getEmail(), kakaoUser.getPassword());
}

Expand Down

0 comments on commit 5c80fc4

Please sign in to comment.