Skip to content

[Feat] 전역 CORS 설정 추가 (#31)#32

Open
seohyun27 wants to merge 1 commit intomainfrom
feat/31-cors
Open

[Feat] 전역 CORS 설정 추가 (#31)#32
seohyun27 wants to merge 1 commit intomainfrom
feat/31-cors

Conversation

@seohyun27
Copy link
Copy Markdown
Contributor

#️⃣ 연관된 이슈

Closes #31

#️⃣ 작업 내용

  • SecurityConfig 내에 전역 CORS 설정을 위한 corsConfigurationSource 빈(Bean) 추가

  • 프론트엔드 접근 허용 도메인 추가:

    • 로컬 환경: http://localhost:3000
    • 배포 환경: https://wagle-client.vercel.app
  • 프론트엔드와의 원활한 JWT 토큰 통신을 위해 allowCredentials(true) 설정 및 Authorization 헤더 노출(exposedHeaders) 설정 추가

  • 허용할 HTTP 메서드 명시 (GET, POST, PUT, PATCH, DELETE, OPTIONS)

#️⃣ 테스트 결과

  • 백엔드 로컬 환경 정상 빌드 및 실행 확인 완료
  • [진행 예정] 해당 PR 브랜치를 Dev 환경에 선배포하여 프론트엔드 측 실제 API 호출(CORS 에러 발생 여부) 테스트 예정

#️⃣ 셀프 체크리스트

  • (Server) 로컬 환경에서 정상적으로 빌드되나요?
  • (Server) 예시 데이터를 통해 테스트를 마쳤나요? (프론트 연동 테스트 대기 중)
  • (Client) 화면이 깨지지 않고 렌더링되나요?
  • Swagger 문서가 최신화되었나요? (API 변경 시) (해당 없음)
  • 코드 컨벤션을 준수했나요?

#️⃣ 리뷰 요구사항

🚨 [중요] 프론트엔드 팀 확인 요청 🚨
본 PR은 머지하기 전에 프론트엔드 측에서 CORS 에러가 완전히 해결되었는지 실제 검증이 필요합니다.

현재 이 PR 브랜치를 기준으로 Dev 서버에 자동 배포(Action)되도록 진행할 예정입니다. 프론트엔드 팀원분들께서는 배포가 완료된 후, 로컬 환경 및 Vercel 배포 환경에서 Dev 서버 API를 호출해 보시고 CORS 에러 없이 정상적으로 데이터를 주고받는지(토큰 포함) 확인 부탁드립니다. 테스트 결과 문제가 없다면 코멘트 남겨주시고 Approve 해주시면 그때 최종적으로 Merge 하겠습니다!

📎 참고 자료 (Optional)

  • 허용된 프론트 배포 주소: https://wagle-client.vercel.app/

Copy link
Copy Markdown
Contributor

@seoyeoki seoyeoki left a comment

Choose a reason for hiding this comment

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

도메인 및 기타 설정 유효합니다. 👍

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feat] 프론트엔드 API 통신을 위한 CORS 정책 설정 (Local & Vercel)

2 participants