Skip to content

Revert "fix: CORS exposed headers에 Content-Type 추가"#495

Merged
dh2906 merged 1 commit intodevelopfrom
revert-493-feat/sse-cors
Apr 1, 2026
Merged

Revert "fix: CORS exposed headers에 Content-Type 추가"#495
dh2906 merged 1 commit intodevelopfrom
revert-493-feat/sse-cors

Conversation

@dh2906
Copy link
Copy Markdown
Contributor

@dh2906 dh2906 commented Apr 1, 2026

Reverts #493

@dh2906 dh2906 self-assigned this Apr 1, 2026
@dh2906 dh2906 merged commit a8153df into develop Apr 1, 2026
3 of 4 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: dacbb60b-412e-489e-a8a0-39fb98e4b504

📥 Commits

Reviewing files that changed from the base of the PR and between 389ae58 and 820a6ad.

📒 Files selected for processing (1)
  • src/main/java/gg/agit/konect/global/config/WebConfig.java

📝 Walkthrough

개요

WebConfig 클래스의 CORS 설정에서 exposedHeaders 목록을 3개 항목(Authorization, Content-Type, Cache-Control)에서 1개 항목(Authorization)으로 축소했습니다. 다른 CORS 설정은 변경되지 않았습니다.

변경사항

병합 / 파일 요약
CORS 설정 업데이트
src/main/java/gg/agit/konect/global/config/WebConfig.java
exposedHeaders에서 Content-TypeCache-Control 제거. Authorization 헤더만 클라이언트에 노출

예상 코드 리뷰 소요 시간

🎯 1 (Trivial) | ⏱️ ~3분

🐰 CORS 헤더들을 정리하니
인가만 남겨두었네
간결한 설정으로
더 깔끔해졌어요~
작은 변화 큰 효과! ✨

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revert-493-feat/sse-cors

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@dh2906 dh2906 deleted the revert-493-feat/sse-cors branch April 1, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant