-
Notifications
You must be signed in to change notification settings - Fork 1
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 로컬 프록시 도메인에 와우클래스 추가 #615
Conversation
Walkthrough변경사항은 클라이언트 URL 상수를 추가하고 CORS 설정을 간소화하며, 로컬 환경을 확인하는 유틸리티 메서드를 추가하여 코드의 유지 관리성을 향상시키는 데 중점을 두었습니다. 이를 통해 개발 및 로컬 환경에 대한 URL 관리가 용이해지고, 보안 설정이 더욱 효율적으로 개선되었습니다. Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
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.
lgtm
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
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
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (3)
- src/main/java/com/gdschongik/gdsc/global/common/constant/UrlConstant.java (2 hunks)
- src/main/java/com/gdschongik/gdsc/global/config/WebSecurityConfig.java (1 hunks)
- src/main/java/com/gdschongik/gdsc/global/util/EnvironmentUtil.java (1 hunks)
Additional comments not posted (4)
src/main/java/com/gdschongik/gdsc/global/util/EnvironmentUtil.java (1)
39-41
: 새로운 메서드 추가가 적절합니다.
isLocalProfile()
메서드는 로컬 환경을 확인하는 데 유용하며, 기존의 프로필 확인 메서드와 일관성을 유지하고 있습니다.src/main/java/com/gdschongik/gdsc/global/common/constant/UrlConstant.java (2)
21-25
: 로컬 클라이언트 URL 상수 추가가 적절합니다.로컬 클라이언트 URL 상수의 추가는 로컬 개발 환경에서의 URL 관리를 용이하게 합니다.
37-48
: URL 목록의 구조 개선이 잘 이루어졌습니다.
LOCAL_CLIENT_URLS
와DEV_AND_LOCAL_CLIENT_URLS
의 추가는 URL 관리의 유연성을 높이고, 코드의 유지 보수성을 향상시킵니다.src/main/java/com/gdschongik/gdsc/global/config/WebSecurityConfig.java (1)
170-175
: CORS 설정의 개선이 잘 이루어졌습니다.환경 프로필에 따른 CORS 설정의 간소화는 코드의 명확성과 유지 보수성을 높입니다. 특히 로컬 프로필에 대한 명시적인 처리가 추가되어 로컬 환경에서의 개발이 용이해졌습니다.
🌱 관련 이슈
📌 작업 내용 및 특이사항
📝 참고사항
📚 기타
Summary by CodeRabbit
New Features
isLocalProfile()
메서드를 추가했습니다.Enhancements