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

[BE-55] feat : 토큰 쿠키에 담아 보내기 #166

Closed
wants to merge 1 commit into from

Conversation

LJH098
Copy link
Contributor

@LJH098 LJH098 commented Feb 24, 2024

개요

close #165

작업사항

  • 엑세스 / 리프레쉬 토큰을 쿠키에 굽기

변경로직

  • 내용을 적어주세요.

reference

- 엑세스 / 리프레쉬 토큰을 쿠키에 굽기
@LJH098 LJH098 added the feature✨ This issue or pull request already exists label Feb 24, 2024
@LJH098 LJH098 self-assigned this Feb 24, 2024
@LJH098 LJH098 had a problem deploying to Recruit-Production February 24, 2024 13:34 — with GitHub Actions Failure
@LJH098 LJH098 changed the title feat : 로그인 [BE-55] feat : 로그인 Feb 24, 2024
@LJH098 LJH098 changed the title [BE-55] feat : 로그인 [BE-55] feat : 토큰 쿠키에 담아 보내기 Feb 24, 2024
Copy link
Collaborator

@BlackBean99 BlackBean99 left a comment

Choose a reason for hiding this comment

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

Reject

TokenResponse tokenResponse = userLoginUseCase.execute(loginRequestDto);
Cookie accessCookie = new Cookie("ACCESSTOKEN", tokenResponse.getAccessToken());
Copy link
Collaborator

Choose a reason for hiding this comment

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

ACCESS_TOKEN으로 바꿔주세요

accessCookie.setMaxAge(60 * 60 * 24 * 30);
response.addCookie(accessCookie);

Cookie refreshCookie = new Cookie("REFRESHTOKEN", tokenResponse.getRefreshToken());
Copy link
Collaborator

Choose a reason for hiding this comment

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

REFRESH_TOKEN으로 바꿔주세요

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature✨ This issue or pull request already exists
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE-55] 토큰 쿠키에 담아서 보내기
2 participants