-
Notifications
You must be signed in to change notification settings - Fork 2
feat: 이벤트 도메인 CORS 설정 #1231
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: 이벤트 도메인 CORS 설정 #1231
Conversation
📝 WalkthroughWalkthrough환경별 이벤트 클라이언트 URL 상수 3종(프로덕션/개발/로컬)을 추가하고, 각 환경의 클라이언트 URL 목록에 해당 이벤트 URL을 포함하도록 수정했습니다. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Job Summary for GradleCheck Style and Test to Develop :: build-test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
🧹 Nitpick comments (1)
src/main/java/com/gdschongik/gdsc/global/common/constant/UrlConstant.java (1)
42-47: DEV_CLIENT_URLS에 이벤트 URL 추가 LGTM세 환경 모두 동형 변경이라 향후 누락 방지를 위해 간단한 테스트(각 환경 리스트에 이벤트 URL 포함 여부 검증) 추가를 권장합니다. 원하시면 테스트 초안 제공 가능합니다.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
src/main/java/com/gdschongik/gdsc/global/common/constant/UrlConstant.java(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: build-test
🔇 Additional comments (5)
src/main/java/com/gdschongik/gdsc/global/common/constant/UrlConstant.java (5)
22-22: DEV 이벤트 도메인 상수 추가 LGTM개발 환경 허용 도메인 확장으로 적절합니다.
28-28: LOCAL 이벤트 도메인 상수 추가 LGTM로컬 HTTPS 인증서 신뢰 및 도메인 해석(hosts/DNS)만 사전 점검해 주세요.
36-40: PROD_CLIENT_URLS에 이벤트 URL 추가 LGTM리스트 일관성 유지되고, 누락 없이 반영되었습니다.
54-54: LOCAL_CLIENT_URLS에 이벤트 URL 추가 LGTMDEV_AND_LOCAL 통합 리스트에도 자연스럽게 포함되므로 추가 작업 불필요합니다.
16-16: 확인 필요: PROD_CLIENT_EVENT_URL이 CORS 허용 목록에 포함되었는지 확인하세요WebSecurityConfig의 corsConfigurationSource()가 setAllowedOriginPatterns(PROD_CLIENT_URLS)를 사용하며, 저장소에서 PROD_CLIENT_EVENT_URL을 직접 참조하는 곳은 발견되지 않았습니다. 이벤트 도메인을 실제로 허용하려면 PROD_CLIENT_URLS에 event.wawoo.dev를 추가하거나 CORS 설정에서 PROD_CLIENT_EVENT_URL을 직접 참조하도록 수정하세요. 운영 배포 전 event.wawoo.dev의 DNS/TLS 유효성 및 LB/프록시의 Origin 전달 여부도 확인하세요.
참고: src/main/java/com/gdschongik/gdsc/global/config/WebSecurityConfig.java (corsConfigurationSource(), ≈179–204)
uwoobeat
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
|
프론트쪽 병목 있을 것 같아서 일단 머지하겠습니다 |
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
📚 기타
Summary by CodeRabbit