-
Notifications
You must be signed in to change notification settings - Fork 4
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
cookie의 path를 /auth에서 /로 변경 및 RefreshToken TTL설정 #610
Merged
Conversation
This file contains 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
jeomxon
changed the title
cookie의 path를 /auth에서 /로 변경
cookie의 path를 /auth에서 /로 변경 및 RefreshToken TTL설정
Sep 14, 2023
aiaiaiai1
approved these changes
Sep 14, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
woo-chang
approved these changes
Sep 14, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다 👍👍
tjdtls690
approved these changes
Sep 14, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ㅡ 문 신 ㅡ
Gilpop8663
added a commit
that referenced
this pull request
Sep 14, 2023
* fix: (#598) 로그인 시 쿠키를 주고 받을 수 있도록 fetch 설정 변경 (#600) * cookie의 path를 /auth에서 /로 변경 및 RefreshToken TTL설정 (#610) * refactor: (#608) cookie의 path를 루트 경로로 지정 * refactor: (#608) refreshtoken을 redis에 저장할 때 ttl설정 * 구글 애널리틱스 작동을 위한 모듈 설치, 게시글 본문의 링크 클릭 가능하도록 구현 (#613) * fix: (#612) react-gtm-module을 react-ga4 로 대체, RouteChangeTracker 컴포넌트 추가 * feat: (#606) 게시글 작성 시 링크 넣기 기능 추가 * feat: (#606) 게시글 본문에 링크 있으면 a 태그로 인식되도록 구현 * chore: (#606) 공지사항 설명 수정 * chore: (#606) 불필요한 주석 삭제 * 로그인, 투표 통계, 회원정보 페이지 lazy import 적용 (#571) * feat: (#556) lazy import 설정 및 트리쉐이킹 설정 * feat: (#556) 로그인, 투표 통계, 회원정보 입력 페이지 lazy import 적용 * feat: (#556) 번들 이름 매번 바뀌도록 변경 및 수정되었던 코드 복구 * feat: (#556) Suspense 코드 복구 --------- Co-authored-by: jero_kang <[email protected]> * 게시글 작성 시 이미지 파일을 보낼 때 webp 로 압축하여 성능 개선 (#614) * feat: (#555) browser-image-compression 설치 및 본문 이미지 훅에 적용 * feat: (#555) 선택지 옵션 사진을 webp로 변환하도록 구현 --------- Co-authored-by: 최우창 <[email protected]> Co-authored-by: Jun-Hyeok Sin <[email protected]> Co-authored-by: jero_kang <[email protected]> Co-authored-by: jeomxon <[email protected]> Co-authored-by: aiaiaiai1 <[email protected]> Co-authored-by: chsua <[email protected]> Co-authored-by: lookh <[email protected]> Co-authored-by: 김영길/KIM YOUNG GIL <[email protected]> Co-authored-by: jero_kang <[email protected]> Co-authored-by: chsua <[email protected]>
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
🔥 연관 이슈
close: #608
📝 작업 요약
/auth
에서/
로 변경했습니다.⏰ 소요 시간
1분
🔎 작업 상세 설명
/auth
로 경로를 설정하게 되면 새로고침 시 쿠키가 삭제됩니다.다른 경로에서 한번 받아오는 것은 되지만 그 이후에는 새로고침을 하면서 경로가 다르다고 판단하여 삭제하는 것 같아요.
/
로 설정해야 새로고침을 해도 쿠키가 남아있습니다.🌟 논의 사항
없음.