Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[FIX] CORS 문제 해결 #460

Merged
merged 11 commits into from
Jan 8, 2025
Merged

[FIX] CORS 문제 해결 #460

merged 11 commits into from
Jan 8, 2025

Conversation

leegwichan
Copy link
Contributor

Issue Number

Closed #459

As-Is

  • 현재 Dev 환경에서 CORS 문제 발생

To-Be

  • 여러 도메인이 허용될 수 있도록 CorsConfig 파일 수정
  • 여러 도메인을 허용하도록 application.yml 파일 수정

Check List

  • 테스트가 전부 통과되었나요?
  • 모든 commit이 push 되었나요?
  • merge할 branch를 확인했나요?
  • Assignee를 지정했나요?
  • Label을 지정했나요?

Test Screenshot

image

(Optional) Additional Description

  • CORS 문제라 빠르게 머지하도록 하겠습니다.

@leegwichan leegwichan added 🛠 fix 버그 🍃 BE back end 🔥 Emergency 긴급 처리(2명 리뷰 머지 PR) labels Jan 5, 2025
@leegwichan leegwichan self-assigned this Jan 5, 2025
@leegwichan leegwichan merged commit 22c403a into develop Jan 8, 2025
2 of 3 checks passed
@leegwichan leegwichan deleted the fix/#459 branch January 8, 2025 05:16
@leegwichan leegwichan restored the fix/#459 branch January 11, 2025 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jhon3242 jhon3242 Awaiting requested review from jhon3242

@PgmJun PgmJun Awaiting requested review from PgmJun

@GIVEN53 GIVEN53 Awaiting requested review from GIVEN53

Assignees

@leegwichan leegwichan

Labels
🍃 BE back end 🔥 Emergency 긴급 처리(2명 리뷰 머지 PR) 🛠 fix 버그
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[FIX] CORS 문제 해결
1 participant
@leegwichan