Skip to content

[FEAT] 쿠키 설정 추가#143

Open
wlqgkrry wants to merge 1 commit into
devfrom
feature/reissue-v1
Open

[FEAT] 쿠키 설정 추가#143
wlqgkrry wants to merge 1 commit into
devfrom
feature/reissue-v1

Conversation

@wlqgkrry
Copy link
Copy Markdown
Collaborator


name: 오지현
about: 쿠키
title: 쿠키 설정 추가
labels: open #142
assignees: 한혜수

📌 작업 내용

  • 쿠키 설정을 추가하였습니다.

🔍 주요 변경 사항

  • auth관련, user관련 파일이 수정되었습니다.
  • 토큰 발급, 토큰 재발급, 토큰 만료 시키는 모든 경우에 동일한 쿠키 설정을 추가하였습니다.

🧪 테스트 결과

  • 직접 실행하여 정상 동작 확인함 ->쿠키 기능을 로컬 환경에서 테스트하지 못하여.. 못했습니다
  • 주요 시나리오에 대한 테스트 완료
  • 예외 상황/경계 조건 확인함 (선택)

📎 관련 이슈

✅ 체크리스트

  • 빌드/테스트 정상 작동 확인
  • 커밋 메시지 규칙 준수 (ex. [FEAT], [FIX], [REFACTOR])
  • 컨벤션 준수 (코드 스타일, 네이밍 등)
  • 불필요한 디버깅 코드, 로그 제거
  • 주석 및 TODO 정리

📣 리뷰어에게 전달할 내용

  • cookieService 부분 보시고 맞는지 확인해주시면 좋을 것 같습니다
  • 쿠키 사용 및 세팅을 처음 해봐서..

@wlqgkrry wlqgkrry requested a review from hyesuhan July 29, 2025 10:11
@wlqgkrry wlqgkrry self-assigned this Jul 29, 2025
@wlqgkrry wlqgkrry added the ✨ FEAT 새로운 기능 추가 label Jul 29, 2025
Copy link
Copy Markdown
Collaborator

@hyesuhan hyesuhan left a comment

Choose a reason for hiding this comment

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

확인했습니다! 다 까끔하게 분리하고 함수들도 규칙있게 적으신거 같아요! 수고하셨습니다


// 리프레시 후에는 accessToken/refreshToken 쿠키 모두 세팅
cookieService.setTokenCookies(response, responseDto.getNewAccessToken(), responseDto.getNewRefreshToken());

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

저는 재발급 = 리프레쉬 토큰 이라고 새각했는데 아니네요! 새 토큰 발급받는 api 였구뇨!

@wlqgkrry wlqgkrry added the ❌NOT TO BE MERGED 당장 병합되면 안 될 pr을 의미합니다. label Jul 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ FEAT 새로운 기능 추가 ❌NOT TO BE MERGED 당장 병합되면 안 될 pr을 의미합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants