🔀 Refactor Review auth

src/main/java/com/mycom/backenddaengplace/auth/config/SecurityConfig.java

@@ -32,45 +32,24 @@ public class SecurityConfig {
     @Bean
     public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
         http
-                // CSRF 비활성화 (REST API 스타일에서는 보통 사용하지 않음)
                 .csrf(AbstractHttpConfigurer::disable)
-                // 폼 로그인 비활성화 (소셜 로그인만 사용)
+                .sessionManagement(session ->
+                        session.sessionCreationPolicy(SessionCreationPolicy.STATELESS))
                 .formLogin(AbstractHttpConfigurer::disable)
-                // HTTP 기본 인증 비활성화
                 .httpBasic(AbstractHttpConfigurer::disable)
 
-                // CORS 설정
-                .cors(corsCustomizer -> corsCustomizer.configurationSource(request -> {
-                    CorsConfiguration configuration = new CorsConfiguration();
-                    configuration.setAllowedOrigins(Collections.singletonList("http://localhost:8080")); // React 프론트엔드 URL
-                    configuration.setAllowedMethods(Collections.singletonList("*")); // 모든 HTTP 메서드 허용
-                    configuration.setAllowCredentials(true); // 쿠키 허용
-                    configuration.setAllowedHeaders(Collections.singletonList("*")); // 모든 헤더 허용
-                    configuration.setMaxAge(3600L); // CORS 캐싱 시간 설정
-                    return configuration;
-                }))
-
-                // 소셜 로그인 설정
-                .oauth2Login(oauth2 -> oauth2
-                        .userInfoEndpoint(userInfo -> userInfo.userService(customOAuth2UserService)) // 사용자 정보 처리
-                        .successHandler(customSuccessHandler)) // 로그인 성공 시 처리
-
-                // JWT 인증 필터 추가
-                .addFilterBefore(jwtFilter, UsernamePasswordAuthenticationFilter.class)
-                // 로그아웃 필터 추가
-                .addFilterBefore(customLogoutFilter, LogoutFilter.class)
-
-                // 권한별 요청 처리
                 .authorizeHttpRequests(auth -> auth
-
-                        // 토큰 재발급, 루트 페이지, 로그인, OAuth2 요청은 인증 없이 접근 가능
-                        .requestMatchers("/reissue", "/", "/login", "/oauth2/**").permitAll()
-
-                        // 나머지 요청은 인증 필요
+                        .requestMatchers("/login", "/oauth2/**", "/auth/**", "/reissue").permitAll()
+                        // "/" 경로 제외
                         .anyRequest().authenticated())
 
-                // 세션 정책: Stateless
-                .sessionManagement(session -> session.sessionCreationPolicy(SessionCreationPolicy.STATELESS));
+                .oauth2Login(oauth2 -> oauth2
+                        .userInfoEndpoint(userInfo ->
+                                userInfo.userService(customOAuth2UserService))
+                        .successHandler(customSuccessHandler))
+                .addFilterBefore(jwtFilter, UsernamePasswordAuthenticationFilter.class) // JWT 필터 추가
+                .addFilterBefore(customLogoutFilter, LogoutFilter.class);
+
 
         return http.build();
     }

src/main/java/com/mycom/backenddaengplace/auth/config/WebConfig.java

@@ -3,7 +3,6 @@
 import com.mycom.backenddaengplace.auth.interceptor.AuthorizationInterceptor;
 import lombok.RequiredArgsConstructor;
 import org.springframework.web.servlet.config.annotation.InterceptorRegistry;
-import org.springframework.web.servlet.config.annotation.ResourceHandlerRegistry;
 import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
 
 @RequiredArgsConstructor
@@ -19,9 +18,9 @@ public void addInterceptors(InterceptorRegistry registry) {
          *              // (...)
          *              .anyRequest().authenticated()) // 그 외의 요청은 인증 필요
          */
-        registry.addInterceptor(authorizationInterceptor)   // 인터셉터 등록 (여러 인터셉터 등록이 가능하며, 필요 시 순서 조정도 가능)
-                .addPathPatterns("/**")                     // 인터셉터 적용 경로
-                .excludePathPatterns("/");                  // 인터셉터 제외 경로
+            registry.addInterceptor(authorizationInterceptor)
+                    .addPathPatterns("/**") // 모든 경로에 적용, '/' 제외 필요 없음
+                    .excludePathPatterns("/error", "/logout", "/api/login"); // 에러 페이지, 로그아웃, 로그인 API는 제외
     }
 
 }

src/main/java/com/mycom/backenddaengplace/auth/dto/CustomOAuth2User.java

@@ -1,49 +1,48 @@
 package com.mycom.backenddaengplace.auth.dto;
 
-
+import com.mycom.backenddaengplace.member.domain.Member;
+import lombok.Getter;
 import org.springframework.security.core.GrantedAuthority;
 import org.springframework.security.oauth2.core.user.OAuth2User;
 
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Map;
 
+@Getter
 public class CustomOAuth2User implements OAuth2User {
 
     private final UserDTO userDTO;
+    private final Member member;  // Member 추가
 
-    public CustomOAuth2User(UserDTO userDTO) {
-
+    public CustomOAuth2User(UserDTO userDTO, Member member) {  // 생성자 수정
         this.userDTO = userDTO;
+        this.member = member;
     }
 
     @Override
     public Map<String, Object> getAttributes() {
-
         return null;
     }
 
     @Override
     public Collection<? extends GrantedAuthority> getAuthorities() {
         Collection<GrantedAuthority> collection = new ArrayList<>();
-        collection.add(new GrantedAuthority() {
-            @Override
-            public String getAuthority() {
-                // 권한이 없으면 기본 ROLE_USER 반환
-                return "ROLE_USER";
-            }
-        });
+        collection.add(() -> "ROLE_USER");  // 람다식으로 단순화
         return collection;
     }
 
     @Override
     public String getName() {
-        // userDTO.getName() 대신 nickname을 반환
         return userDTO.getNickname() != null ? userDTO.getNickname() : "Unknown User";
     }
 
     public String getUsername() {
-        // userDTO.getUsername() 대신 provider + providerId 반환
         return userDTO.getProvider() + "_" + userDTO.getProviderId();
     }
+
+    // Member 객체를 반환하는 메소드 추가
+    public Member getMember() {
+        return member;
+    }
 }

src/main/java/com/mycom/backenddaengplace/auth/dto/CustomUserDetails.java

@@ -1,17 +1,19 @@
 package com.mycom.backenddaengplace.auth.dto;
 
 import com.mycom.backenddaengplace.member.domain.Member;
+import lombok.Getter;
 import lombok.RequiredArgsConstructor;
 import org.springframework.security.core.GrantedAuthority;
 import org.springframework.security.core.userdetails.UserDetails;
 
 import java.util.ArrayList;
 import java.util.Collection;
 
+@Getter
 @RequiredArgsConstructor
 public class CustomUserDetails implements UserDetails {
 
-    private final Member Member;
+    private final Member member;
 
     @Override
     public Collection<? extends GrantedAuthority> getAuthorities() {
@@ -35,7 +37,7 @@ public String getPassword() {
     @Override
     public String getUsername() {
         // 사용자 이름 대신 providerId를 고유 식별자로 반환
-        return Member.getProvider() + "_" + Member.getProviderId();
+        return member.getProvider() + "_" + member.getProviderId();
     }
 
     @Override

src/main/java/com/mycom/backenddaengplace/auth/jwt/JWTFilter.java

@@ -1,6 +1,9 @@
 package com.mycom.backenddaengplace.auth.jwt;
 
+import com.mycom.backenddaengplace.auth.dto.CustomOAuth2User;
 import com.mycom.backenddaengplace.auth.dto.CustomUserDetails;
+import com.mycom.backenddaengplace.auth.dto.UserDTO;
+import com.mycom.backenddaengplace.auth.repository.AuthMemberRepository;
 import com.mycom.backenddaengplace.member.domain.Member;
 import io.jsonwebtoken.ExpiredJwtException;
 import jakarta.servlet.FilterChain;
@@ -25,85 +28,72 @@
 public class JWTFilter extends OncePerRequestFilter {
 
     private final JWTUtil jwtUtil;
+    private final AuthMemberRepository authMemberRepository; // 추가
 
     @Override
     protected void doFilterInternal(HttpServletRequest request, HttpServletResponse response, FilterChain filterChain) throws ServletException, IOException {
-        // 헤더에서 access키에 담긴 토큰을 꺼냄
-        // String accessToken = request.getHeader("access");
-        // String accessToken = request.getHeader("Authorization");
         String accessToken = request.getHeader(HttpHeaders.AUTHORIZATION);
         log.info("{}{} {}{}Authorization: {}", System.lineSeparator(), request.getMethod(), request.getRequestURI(),
                 System.lineSeparator(), accessToken);
 
-        // 토큰이 없다면 다음 필터로 넘김
-        if (accessToken == null) {
+        if (accessToken == null || !accessToken.startsWith("Bearer ")) {
             filterChain.doFilter(request, response);
             return;
         }
 
-        // 토큰 만료 여부 확인, 만료시 다음 필터로 넘기지 않음
+        accessToken = accessToken.substring(7); // "Bearer " 제거
+
         try {
             jwtUtil.isExpired(accessToken);
         } catch (ExpiredJwtException e) {
-
-            //response body
             PrintWriter writer = response.getWriter();
             writer.print("access token expired");
-
-            //response status code
             response.setStatus(HttpServletResponse.SC_UNAUTHORIZED);
             return;
         }
 
-        // 토큰이 access인지 확인 (발급시 페이로드에 명시)
         String category = jwtUtil.getCategory(accessToken);
-
         if (!category.equals("access")) {
-
-            //response body
             PrintWriter writer = response.getWriter();
             writer.print("invalid access token");
-
-            //response status code
             response.setStatus(HttpServletResponse.SC_UNAUTHORIZED);
             return;
         }
 
-// username, role 값을 획득
         String username = jwtUtil.getUsername(accessToken);
         String role = jwtUtil.getRole(accessToken);
 
-// Member 객체 생성 (빌더 패턴 사용)
-        Member member = Member.builder()
-                .provider(username)
-                .providerId(username)
-                .nickname(username)
-                .build();
-
-
-// CustomUserDetails 생성
-        /*
-         * TODO: 인증된 사용자 정보를 만드는 방법
-         *  1. User 클래스를 사용하는 방법
-         *  2. JPA 엔티티에 UserDetails 인터페이스를 구현한 클래스를 사용하는 방법
-         *  3. 커스텀 UserDetails 클래스를 만들어 사용하는 방법 (추천)
-         */
-        // User customUserDetails = new User(Member.getEmail(), Member.getPassword(), List.of(authEntity.getRole()));
-        CustomUserDetails customUserDetails = new CustomUserDetails(member);
-
-// Authentication 객체 생성
-        Authentication authToken = new UsernamePasswordAuthenticationToken(
-                customUserDetails,
-                null,
-                customUserDetails.getAuthorities() // 기본 권한 ROLE_USER 반환
-        );
-
-// SecurityContextHolder에 인증 정보 설정
-        // SecurityContext, PersistenceContext, XxxContext: Xxx
-        SecurityContextHolder.getContext().setAuthentication(authToken);
-
-// 다음 필터로 요청 전달
-        filterChain.doFilter(request, response);
+        // username에서 provider와 providerId 추출
+        String[] parts = username.split("_");
+        if (parts.length == 2) {
+            String provider = parts[0];
+            String providerId = parts[1];
+
+            // DB에서 Member 조회  String[] parts = username.split("_");
+            Member member = authMemberRepository.findByProviderAndProviderId(provider, providerId);
+            if (member != null) {
+                // UserDTO 생성
+                UserDTO userDTO = new UserDTO();
+                userDTO.setProvider(provider);
+                userDTO.setProviderId(providerId);
+                userDTO.setEmail(member.getEmail());
+                userDTO.setNickname(member.getNickname());
+                userDTO.setProfileImage(member.getProfileImageUrl());
+
+                // CustomOAuth2User 생성
+                CustomOAuth2User customOAuth2User = new CustomOAuth2User(userDTO, member);
+
+                // Authentication 객체 생성 및 SecurityContext에 설정
+                Authentication authToken = new UsernamePasswordAuthenticationToken(
+                        customOAuth2User,
+                        null,
+                        customOAuth2User.getAuthorities()
+                );
+
+                SecurityContextHolder.getContext().setAuthentication(authToken);
+            }
+        }
 
+        filterChain.doFilter(request, response);
     }
 }