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.

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

Interceptor의 Preflight 처리 로직 개선 #223

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

ksk0605
Copy link
Contributor

@ksk0605 ksk0605 commented Aug 5, 2024

PR의 목적이 무엇인가요?

CorsUtils 를 사용하여 Interceptor의 Preflight 처리 로직 개선

이슈 ID는 무엇인가요?

설명

스프링은 org.springframework.web.cors 패키지의 CorsUtils 라는 클래스를 제공하여
Cors 관련 설정에 도움이 되는 도구를 제공하고 있습니다.

기존에 복잡하게 작동하던 preflight 인증 여부 검증 메소드를 CorsUtils 클래스를 사용하여 개선하였습니다.

질문 혹은 공유 사항 (Optional)

@ksk0605 ksk0605 added BE 백엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정) labels Aug 5, 2024
@ksk0605 ksk0605 added this to the 3차 스프린트 milestone Aug 5, 2024
@ksk0605 ksk0605 self-assigned this Aug 5, 2024
Copy link
Contributor

@ay-eonii ay-eonii left a comment

Choose a reason for hiding this comment

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

굉장히 깔끔해졌네요 👍 고생하셨습니다~!

Copy link
Contributor

@pricelees pricelees left a comment

Choose a reason for hiding this comment

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

굿👍👍

@ksk0605 ksk0605 merged commit 0c9a931 into develop-backend Aug 6, 2024
1 check passed
@hoyeonyy hoyeonyy deleted the refactor/#222 branch August 7, 2024 04:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈입니다. ⚒️ 리팩터링 refactor (기능이 변경되지는 않지만 코드를 수정)
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

3 participants