Skip to content

Refactor/#326 samesite principel change#328

Merged
Jjiggu merged 2 commits intodevelopfrom
refactor/#326-samesite-principel-change
Nov 9, 2025
Merged

Refactor/#326 samesite principel change#328
Jjiggu merged 2 commits intodevelopfrom
refactor/#326-samesite-principel-change

Conversation

@Jjiggu
Copy link
Contributor

@Jjiggu Jjiggu commented Nov 9, 2025

작업 요약

yml 변경

Issue Link

#326

문제점 및 어려움

해결 방안

Reference

Summary by CodeRabbit

릴리스 노트

이 버전에는 최종 사용자에게 표시되는 변경 사항이 없습니다.

  • 스타일
    • 코드 포맷팅 개선

@Jjiggu Jjiggu self-assigned this Nov 9, 2025
@Jjiggu Jjiggu added the bug Something isn't working label Nov 9, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 9, 2025

Caution

Review failed

The pull request is closed.

요약

CorsConfig.java 파일의 반환 문 뒤에 두 개의 후행 공백 줄을 추가했습니다. 로직이나 설정에는 변경 사항이 없습니다.

변경 사항

응집도 / 파일 변경 요약
코드 스타일
nowait-app-user-api/src/main/java/com/nowait/applicationuser/config/security/CorsConfig.java
CorsConfigurationSource 빈의 반환 문 뒤에 후행 공백 줄 추가

예상 코드 리뷰 노력도

🎯 1 (Trivial) | ⏱️ ~2분

제안 라벨

refactor

제안 검토자

  • HyemIin
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/#326-samesite-principel-change

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7708cc6 and 6a91ba6.

📒 Files selected for processing (1)
  • nowait-app-user-api/src/main/java/com/nowait/applicationuser/config/security/CorsConfig.java (1 hunks)

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.

@github-actions github-actions bot requested a review from HyemIin November 9, 2025 11:02
@Jjiggu Jjiggu merged commit 3a112e4 into develop Nov 9, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant