Conversation
… feature/24-review-api
feat: 마이페이지 관련 API 구현
프론트엔드와 API가 서로 다른 도메인에서 동작하므로 CORS 설정을 추가한다. 쿠키 기반 토큰 재발급을 위해 allowCredentials를 활성화하고, 허용 오리진은 환경변수로 주입받는다.
배포 환경에서는 크로스 사이트 쿠키 전송을 위해 SameSite=None이 필요하고 로컬에서는 Lax가 적합하므로 환경변수로 분리한다. 리버스 프록시 뒤에서 https 스킴을 인식하도록 forward-headers-strategy를 추가한다.
…fication feat: 일정 담당자 지정 알림 생성 연동
refactor: 일정 API 인증 사용자 조회 방식 변경
Updated README.md to provide detailed project information, team members, tech stack, local execution instructions, and collaboration guidelines.
docs: README 업데이트
refactor: 피드백/답글 작성자 인증 방식 통일
csrf().disable() 상태에서 재발급 API는 쿠키만으로 인증되므로 CSRF에 노출된다. CORS 허용 목록은 응답 읽기만 막고, Origin 헤더가 없는 요청은 CorsFilter를 그대로 통과하기 때문에 SameSite=None 환경에서는 크로스 사이트 요청에도 쿠키가 실린다. POST /api/v1/auth/refresh에 대해 Origin(없으면 Referer)이 허용 오리진 또는 API 자신의 오리진과 일치하는지 검증하고, 판별할 수 없으면 403으로 차단한다.
Secure와 SameSite가 별도 환경변수로 분리되어 있어 Secure 없는 SameSite=None 쿠키가 만들어질 수 있다. 브라우저는 이런 쿠키를 저장·전송하지 않으므로 크로스 사이트 재발급이 조용히 실패한다. SameSite=None이면 secure=true를 강제하고, Lax/Strict/None 외의 값도 거부해 잘못된 설정이 기동 단계에서 드러나게 한다.
…tps-config SecurityConfig 충돌 해소. develop의 게스트 피드백/답글 permitAll 규칙과 이 브랜치의 CORS 설정·CSRF 필터 등록을 모두 유지한다.
feat: HTTPS 도메인 전환을 위한 CORS 설정 및 쿠키 SameSite 환경변수화
…rvice feat: 알림 생성 공통 서비스 구현
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
chazy-d
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🔗 관련 이슈 (Related Issue)
📝 작업 내용
develop브랜치를main에 병합해 운영에 배포합니다. 이 PR에서 새로 작성한 코드는 없고, 아래 PR들이 함께 나갑니다.✅ PR 체크리스트