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

feat: 쿠키 설정 추가 #65

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

3Juhwan
Copy link
Member

@3Juhwan 3Juhwan commented Feb 9, 2025

작업 내용

쿠키에 아무런 설정이 되어 있지 않아서, 추가해요. 추가한 설정은 다음과 같아요.

  • httpOnly
  • secure
  • domain
  • path
  • sameSite
  • maxAge

httponly, secure,samesite는 기본으로 설정했고, path는 최상단 경로인 /, 만료 시간은 영구에 가깝게 6년을 설정했어요. 6년이면 신입생이 졸업하고 남지 않을까요? ㅎㅎ

@3Juhwan 3Juhwan requested a review from boyekim February 9, 2025 06:30
@3Juhwan 3Juhwan self-assigned this Feb 9, 2025
@3Juhwan 3Juhwan linked an issue Feb 9, 2025 that may be closed by this pull request
Copy link

github-actions bot commented Feb 9, 2025

Test Results

59 tests   46 ✅  3s ⏱️
17 suites   0 💤
17 files    13 ❌

For more details on these failures, see this check.

Results for commit 11cbd8b.

@3Juhwan
Copy link
Member Author

3Juhwan commented Feb 9, 2025

테스트가 많이 깨지네요. 토큰이 필요한 로직에서 토큰을 전달하지 않아서 그런 것 같아요.
테스트마다 토큰을 설정해 줄지, 토큰을 우회해서 전달하지 않고 테스트 실행할지, 그런 방법을 고민해 봐야 할 것 같아요.

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