[refact] 리프레시 토큰 발급 로직 수정 #186
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
suhhyun524
requested review from
hyunihs,
mushroom1324,
haen-su and
mirageoasis
as code owners
suhhyun524
requested review from
letskuku,
yoonsseo,
itsme-shawn and
YoungGyo-00
as code owners
yoonsseo
approved these changes
Aug 13, 2024
Comment on lines
44
to
45
| String device = request.getHeader("User-Agent").contains("mobile") ? "mobile" : "web"; | ||
| return adminService.signIn(device, signInRequest); |
There was a problem hiding this comment.
enum 쓰는게 나을 듯 싶긴한데.... 생각해보면 여기서 밖에 안쓰이니 굳이 안넣어도 되겠군요
There was a problem hiding this comment.
enum에 대한 의견은 "mobile", "web" 값을 관리하기 위함인가요? 그렇다면 말씀해주셨던 것처럼 여기서만 쓰이고 있으니 상수로 관리해주는 것도 괜찮은 방법일 것 같습니다!
There was a problem hiding this comment.
사용 범위가 좁아서 그냥 문자열로 사용했는데 상수로 관리하는 것도 좋은 것 같네요!
바꾸겠습니다🥰 좋은 의견 감사해용
letskuku
approved these changes
Aug 13, 2024
Comment on lines
44
to
45
| String device = request.getHeader("User-Agent").contains("mobile") ? "mobile" : "web"; | ||
| return adminService.signIn(device, signInRequest); |
There was a problem hiding this comment.
enum에 대한 의견은 "mobile", "web" 값을 관리하기 위함인가요? 그렇다면 말씀해주셨던 것처럼 여기서만 쓰이고 있으니 상수로 관리해주는 것도 괜찮은 방법일 것 같습니다!
hyunihs
added a commit
that referenced
this pull request
Aug 14, 2024
* [feat] 지원 서류 삭제 기능 (#177) * [feat] 지원 서류 삭제 * [chore] update CODEOWNERS * [feat] 지원서 목록 보기 API required false인 parameter들에 대해 default value 설정 * [test] 지원서 목록 보기 API required false인 parameter들에 대한 테스트 코드 작성 * [feat] 지원서 목록 보기 API enum 타입인 parameter들에 대해 allowable values 설정 * [test] 지원서 목록 보기 API enum 타입인 parameter들에 대해 테스트 코드 작성 * [fix] 지원서 엑셀 파일 생성 시각 API 테스트 status code 수정 * [revert] 코드 수정 내역 되돌림 * [feat] 지원서 목록 보기 API required false인 parameter들에 대해 default value 설정 * [test] 지원서 목록 보기 API required false인 parameter들에 대한 테스트 코드 작성 * [feat] 지원서 목록 보기 API enum 타입인 parameter들에 대해 allowable values 설정 * [test] 지원서 목록 보기 API enum 타입인 parameter들에 대해 테스트 코드 작성 * [fix] 지원서 엑셀 파일 생성 시각 API 테스트 status code 수정 * [revert] 코드 수정 내역 되돌림 * [feat] 지원서 목록 보기 API required false인 parameter들에 대해 default value 설정 * [test] 지원서 목록 보기 API required false인 parameter들에 대한 테스트 코드 작성 * [feat] 지원서 목록 보기 API enum 타입인 parameter들에 대해 allowable values 설정 * [test] 지원서 목록 보기 API enum 타입인 parameter들에 대해 테스트 코드 작성 * [fix] 지원서 엑셀 파일 생성 시각 API 테스트 status code 수정 * [feat] 면접 참여 여부 확인용 DTO 생성 #180 * [feat] 최종 활동 여부 확인용 DTO 생성 #180 * [feat] 불가능 사유 unableReason 필드 및 해당 필드 업데이트 로직 추가 #180 * [feat] 최종 활동 여부 확인용 DTO 생성 #180 * [feat] 면접 참여 가능 여부 확인과 활동 가능 여부 확인 Service 로직 작성 #180 * [feat] 활동 가능 여부 확인 시 최종 합격 여부 검증하는 validator 작성 #180 * feat: 면접 참여 및 활동 가능 여부 확인용 Controller 작성 * feat: 면접 참여 및 활동 가능 여부 확인용 Controller 작성 #180 * [refact] 회고록 목록 보기 response dto 필드 수정 * [refact] 회고록 상세 보기/수정 response dto 필드 수정 * [refact] 리프레시 토큰 발급 로직 수정 (#186) * [refact] 리프레시토큰 발급 로직 수정 #185 * [refact] 상수 추가 --------- Co-authored-by: yoonsseo <[email protected]> Co-authored-by: letskuku <[email protected]> Co-authored-by: Yujeong Lee <[email protected]> Co-authored-by: 이윤서 <[email protected]> Co-authored-by: suhhyun <[email protected]>
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.
💡 PR Summary - 리프레시 토큰 발급 로직 수정
📝 Description
User-Agent 헤더를 이용해 웹/모바일 환경을 나눠 리프레시 토큰을 발급하도록 수정하였습니다.
모바일/PC 동시 로그인이 가능해집니다.
🌲 Working Branch
refact/token
📖 Related Issues
이슈 번호