Skip to content

Conversation

@GoGradually
Copy link
Collaborator

변경된 점

  • gateway의 CORS 적용으로 인한 cors 설정 제거

Copilot AI review requested due to automatic review settings December 31, 2025 09:53
@GoGradually GoGradually merged commit d6f2a22 into master Dec 31, 2025
5 checks passed
@GoGradually GoGradually deleted the fix/cors-설정-제거 branch December 31, 2025 09:53
Copy link

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은 게이트웨이 레벨에서 CORS 처리를 통합함에 따라 애플리케이션 레벨의 중복된 CORS 설정을 제거합니다.

  • CORS 관련 설정 파일 및 속성 클래스 제거
  • SecurityConfig에서 CORS 설정 빈 및 관련 import 문 삭제
  • application.yml에서 CORS 설정 제거

Reviewed changes

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

File Description
src/main/resources/application.yml CORS 관련 설정 속성(allowed-origins, allowed-methods 등) 제거
src/main/java/me/pinitnotification/infrastructure/authenticate/config/SecurityConfig.java CORS 설정 빈 메서드 및 관련 import 문 삭제
src/main/java/me/pinitnotification/infrastructure/authenticate/config/CorsProperties.java CORS 속성을 관리하던 설정 클래스 파일 완전 삭제

중요 참고사항: 코드 리뷰 중 SecurityConfig.java의 line 31에서 .cors(Customizer.withDefaults())를 사용하고 있는데, 이 PR에서 corsConfigurationSource 빈을 제거하고 있습니다. 게이트웨이에서 CORS를 완전히 처리한다면 해당 라인을 .cors(AbstractHttpConfigurer::disable)로 변경하여 Spring Security의 CORS 처리를 비활성화하는 것을 권장드립니다. 이 라인은 diff 영역 밖에 있어 공식 코멘트를 남길 수 없었으나, 배포 전 확인이 필요한 중요한 사항입니다.

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