Skip to content

Conversation

@GoGradually
Copy link
Collaborator

변경된 점

  • 스프링 시큐리티 의존성 제거
  • JWT 관련 의존성 제거

Copilot AI review requested due to automatic review settings December 31, 2025 10:18
@GoGradually GoGradually merged commit 55e5da6 into master Dec 31, 2025
3 checks passed
@GoGradually GoGradually deleted the refactor/스프링-시큐리티-의존성-제거 branch December 31, 2025 10:18
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

이 PR은 스프링 시큐리티와 JWT 관련 의존성을 제거하여 인증 방식을 변경합니다. JWT 토큰 기반 인증에서 헤더 기반 인증(X-Member-Id)으로 전환하며, 인증 책임을 외부 서비스(예: API Gateway)로 위임하는 아키텍처 변경을 반영합니다.

  • JWT 토큰 검증 및 파싱 로직 제거
  • Spring Security 필터 체인 및 설정 제거
  • 인증 관련 빈 및 프로바이더 클래스 삭제

Reviewed changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
build.gradle Spring Security 및 JWT(jjwt) 의존성 제거
src/main/resources/application-prod.yml JWT 공개키 경로 설정 제거
src/main/resources/application-dev.yml JWT 공개키 경로 설정 제거
src/test/java/me/gg/pinit/pinittask/PinitTaskApplicationTests.java JwtTokenProvider 목 빈 제거 및 관련 import 정리
src/main/java/me/gg/pinit/pinittask/infrastructure/web/MemberIdArgumentResolver.java JwtTokenProvider 의존성 제거, 직접 헤더에서 Member ID 추출
src/main/java/me/gg/pinit/pinittask/infrastructure/authenticate/SecurityConfig.java Spring Security 설정 파일 전체 삭제
src/main/java/me/gg/pinit/pinittask/infrastructure/authenticate/RsaKeyProvider.java RSA 공개키 로드 유틸리티 클래스 삭제
src/main/java/me/gg/pinit/pinittask/infrastructure/authenticate/JwtTokenProvider.java JWT 토큰 검증 및 파싱 클래스 삭제
src/main/java/me/gg/pinit/pinittask/infrastructure/authenticate/JwtAuthenticationToken.java JWT 인증 토큰 클래스 삭제
src/main/java/me/gg/pinit/pinittask/infrastructure/authenticate/JwtAuthenticationProvider.java JWT 인증 프로바이더 클래스 삭제
src/main/java/me/gg/pinit/pinittask/infrastructure/authenticate/JwtAuthenticationFilter.java JWT 인증 필터 클래스 삭제

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants