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

SecurityFilterChain을 거치는 POST 요청을 처리하기 위한 CORS 설정 추가. #233

Merged
merged 2 commits into from
Nov 3, 2023
Merged

SecurityFilterChain을 거치는 POST 요청을 처리하기 위한 CORS 설정 추가. #233

merged 2 commits into from
Nov 3, 2023

Conversation

xGreenNarae
Copy link
Contributor

작업 내용

  • 기존의 WebMvcConfigurer를 구현하는 CorsRegistry 설정만 가지고서는 SecurityFilterChain을 거치는 GET 이외의 HTTP METHOD Request에서 JS의 Fetch API에 CORS 오류를 발생시킵니다. 기존의 설정을 유지한 채로, SecurityFilterChain에 http.cors(); 를 추가해도 해결이 되고, 애초에 CORS 설정은 보안과 관련된 정책이기 때문에 SecurityConfiguration이라는 이름에 어울린다고 생각해서 이쪽으로 설정을 옮겼습니다.

공유하고 싶은 내용

  • 이제서야, CORS 정책에 대해서 조금 감을 잡았습니다. (일단 문제해결을 위한 처리 방법들의 동작에 대해서 + 원인과 취지에 대한 느낌)
  • 정리본 참고

Close #232

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 3, 2023

Test Coverage Report

Overall Project 66.12% 😞
Files changed 100% 😍

File Coverage
SecurityConfig.java 87.21% 😍

@xGreenNarae xGreenNarae requested a review from jminkkk November 3, 2023 22:09
@xGreenNarae xGreenNarae merged commit 710fb56 into develop Nov 3, 2023
2 checks passed
@xGreenNarae xGreenNarae deleted the fix/#232 branch November 3, 2023 22:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jminkkk jminkkk Awaiting requested review from jminkkk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BE] SecurityFilterChain에 CORS설정 추가
1 participant
@xGreenNarae