🚀 [Deploy] - 디스코드 소셜 로그인 이슈 해결 반영 (#210)

* 🐛 [Fix] - Discord 소셜 로그인 전환 nickname 이슈 해결 (#205) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * 🐛 [Fix] - Discord 소셜 로그인 전환 (#207) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * 🐛 [Fix] - Discord 소셜 로그인 전환 nickname 이슈 해결 #51 (#209) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Chore: 프론트엔드 redirect url 수정 * Fix: 디스코드 소셜 로그인 이슈 해결
9oormthon-univ · Jul 23, 2024 · 0cb40b3 · 0cb40b3
1 parent 3f28f74
commit 0cb40b3
Show file tree

Hide file tree

Showing 7 changed files with 19 additions and 17 deletions.
diff --git a/src/main/java/goormthon/team28/startup_valley/domain/User.java b/src/main/java/goormthon/team28/startup_valley/domain/User.java
@@ -37,9 +37,9 @@ public class User {
     private EProvider provider;
 
     /*  사용자 이용 정보  */
-    @Column(name = "nickname", nullable = false)
+    @Column(name = "nickname")
     private String nickname;
-    @Column(name = "profile_image", nullable = false)
+    @Column(name = "profile_image")
     @Enumerated(EnumType.STRING)
     private EProfileImage profileImage;
     @Column(name = "refresh_Token")

diff --git a/src/main/java/goormthon/team28/startup_valley/repository/UserRepository.java b/src/main/java/goormthon/team28/startup_valley/repository/UserRepository.java
@@ -21,6 +21,7 @@ public interface UserRepository extends JpaRepository<User, Long> {
     Optional<User> findById(Long userId);
     Optional<User> findBySerialId(String serialId);
     boolean existsBySerialId(String serialId);
+    boolean existsByIdAndNicknameIsNull(Long id);
 
     interface UserSecurityForm {
         Long getId();

diff --git a/src/main/java/goormthon/team28/startup_valley/security/config/SecurityConfig.java b/src/main/java/goormthon/team28/startup_valley/security/config/SecurityConfig.java
@@ -30,8 +30,6 @@ public class SecurityConfig {
 
     private final JwtUtil jwtUtil;
     private final JwtAuthenticationManager jwtAuthenticationManager;
-    private final DefaultSuccessHandler defaultSuccessHandler;
-    private final DefaultFailureHandler defaultFailureHandler;
     private final Oauth2SuccessHandler oauth2SuccessHandler;
     private final Oauth2FailureHandler oauth2FailureHandler;
     private final CustomOauth2UserDetailService customOauth2UserDetailService;
@@ -50,20 +48,10 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
                 )
                 .authorizeHttpRequests(request ->
                         request
-                                .requestMatchers("/login").permitAll()
-                                .requestMatchers("/api/auth/**").permitAll()
                                 .requestMatchers("/api/oauth2/sign-up").permitAll()
                                 .requestMatchers("/api/**").hasAnyRole("USER")
                                 .anyRequest().authenticated()
                 )
-                .formLogin(login -> login
-                        .loginPage("/login")
-                        .loginProcessingUrl("/api/auth/sign-in")
-                        .usernameParameter("serial_id")
-                        .passwordParameter("password")
-                        .successHandler(defaultSuccessHandler)
-                        .failureHandler(defaultFailureHandler)
-                )
                 .oauth2Login(login -> login
                         .successHandler(oauth2SuccessHandler)
                         .failureHandler(oauth2FailureHandler)
@@ -84,7 +72,6 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
                 .addFilterBefore(
                         new JwtExceptionFilter(), JwtAuthenticationFilter.class
                 )
-
                 .getOrBuild();
     }
 }
diff --git a/...ain/java/goormthon/team28/startup_valley/security/handler/login/Oauth2FailureHandler.java b/...ain/java/goormthon/team28/startup_valley/security/handler/login/Oauth2FailureHandler.java
@@ -5,12 +5,14 @@
 import jakarta.servlet.ServletException;
 import jakarta.servlet.http.HttpServletRequest;
 import jakarta.servlet.http.HttpServletResponse;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.security.core.AuthenticationException;
 import org.springframework.security.web.authentication.AuthenticationFailureHandler;
 import org.springframework.stereotype.Component;
 
 import java.io.IOException;
 
+@Slf4j
 @Component
 public class Oauth2FailureHandler implements AuthenticationFailureHandler {
 
@@ -20,6 +22,7 @@ public void onAuthenticationFailure(
             HttpServletResponse response,
             AuthenticationException exception
     ) throws IOException, ServletException {
+        log.info("소셜 로그인 실패");
         AuthenticationResponse.makeFailureResponse(response, ErrorCode.INTERNAL_SERVER_ERROR);
     }
 }
diff --git a/...ain/java/goormthon/team28/startup_valley/security/handler/login/Oauth2SuccessHandler.java b/...ain/java/goormthon/team28/startup_valley/security/handler/login/Oauth2SuccessHandler.java
@@ -23,6 +23,13 @@ public class Oauth2SuccessHandler implements AuthenticationSuccessHandler {
 
     @Value("${server.domain}")
     private String domain;
+
+    @Value("${server.sign-up-redirect-url}")
+    private String signUpRedirectUrl;
+
+    @Value("${server.home-redirect-url}")
+    private String homeRedirectUrl;
+
     private final JwtUtil jwtUtil;
     private final UserRepository userRepository;
 
@@ -40,6 +47,9 @@ public void onAuthenticationSuccess(
 
         AuthenticationResponse.makeLoginSuccessResponse(response, domain, jwtTokenDto, jwtUtil.getRefreshExpiration());
 
-        response.sendRedirect("https://" + domain);
+        if (userRepository.existsByIdAndNicknameIsNull(principal.getUserId()))
+            response.sendRedirect(signUpRedirectUrl); // 최초 로그인
+        else
+            response.sendRedirect(homeRedirectUrl);
     }
 }
diff --git a/.../java/goormthon/team28/startup_valley/security/service/CustomOauth2UserDetailService.java b/.../java/goormthon/team28/startup_valley/security/service/CustomOauth2UserDetailService.java
@@ -30,6 +30,7 @@ public class CustomOauth2UserDetailService extends DefaultOAuth2UserService {
     public OAuth2User loadUser(
             OAuth2UserRequest userRequest
     ) throws OAuth2AuthenticationException {
+        log.info("Oauth2UserRequest 진입, userRequest = {}", userRequest);
         // provider 가져오기
         EProvider provider = EProvider.valueOf(
                 userRequest.getClientRegistration().getRegistrationId().toUpperCase()

diff --git a/startup-valley-properties b/startup-valley-properties