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: blackList구현 #103

Merged
merged 32 commits into from
Nov 11, 2024

Conversation

youcastle03
Copy link
Contributor

블랙리스트 기능을 구현하였습니다

@@ -161,17 +161,17 @@ public HttpHeaders setCookie(TokenResponseDTO tokenResponseDTO){
.httpOnly(true)
.secure(true)
.path("/")
.maxAge(3600)
.maxAge(3600*24*14)
.domain(".aeat.jeje.work")
Copy link
Contributor

Choose a reason for hiding this comment

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

.jeje.work로 수정 부탁 드립니다.

  • 인증 방식에 인증 헤더 대신 쿠키 사용 예정인가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

쿠키 사용 안할 것 같습니다

Comment on lines 52 to 57
# redirect_url: https://${FRONTEND_DOMAIN}/login/redirect
redirect_url: http://localhost:8080/api/users/callback
auth_url: https://kauth.kakao.com/oauth/authorize
logout_url: https://kauth.kakao.com/oauth/logout
logout_redirect_url: https://${FRONTEND_DOMAIN}/logout/redirect
# logout_redirect_url: https://${FRONTEND_DOMAIN}/logout/redirect
logout_redirect_url: http://localhost:8080/api/users/logoutWithKakao/callback
Copy link
Contributor

Choose a reason for hiding this comment

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

엔드포인트 완전히 바뀐건가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

아 테스트 한다고 바꿔놨다가 안바꿨네요 수정하겠습니다.

jjh4450
jjh4450 previously approved these changes Nov 11, 2024
@jjh4450 jjh4450 merged commit 6e158a6 into kakao-tech-campus-2nd-step3:weekly/10 Nov 11, 2024
1 check passed
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.

3 participants