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: Refresh Token 추가 #128

Merged
merged 5 commits into from
Feb 6, 2024
Merged

feat: Refresh Token 추가 #128

merged 5 commits into from
Feb 6, 2024

Conversation

EUNCHAEv1006
Copy link
Member

테스트 코드는 모두 통과하였으나, 로컬 환경에서 토큰 조회 시 "Refresh Token을 찾을 수 없습니다."라는 오류가 발생하고 있습니다.

  1. 토큰 생성 로직: 토큰을 생성하고 이를 데이터베이스에 저장하는 로직을 구현하였습니다.
  2. 토큰 조회 로직: 저장된 토큰을 조회하는 로직을 구현하였습니다.
  3. 테스트 코드: 토큰 생성 및 조회 기능에 대한 테스트 코드를 작성하였습니다. 모든 테스트 코드가 성공적으로 통과하였습니다.

로컬 환경에서 발생하는 문제를 해결하기 위해 여러 가지 방법을 시도해보았으나, 아직 문제는 해결되지 않았습니다. 이 PR을 통해 다른 환경에서의 테스트 결과를 확인하고, 다른 팀원들의 피드백을 받고자 합니다.

@EUNCHAEv1006 EUNCHAEv1006 added the enhancement New feature or request label Feb 3, 2024
@EUNCHAEv1006 EUNCHAEv1006 added this to the 유저 API 기능 구현 milestone Feb 3, 2024
@EUNCHAEv1006 EUNCHAEv1006 self-assigned this Feb 3, 2024
Copy link
Contributor

@wkdehdgk159 wkdehdgk159 left a comment

Choose a reason for hiding this comment

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

최종제출까지 꺾이지 않는 열정 최고입니다!

@AllArgsConstructor
@NoArgsConstructor
@Builder
@Getter
Copy link
Contributor

Choose a reason for hiding this comment

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

@DaTa 있는데 @Getter도 있네요? 당신은 욕심쟁이 우후훗?

@Modifying
@Transactional
@Query("update Token t set t.expired = true, t.revoked = true where t.refreshToken = :token")
void setExpiredAndRevoked(String token);
Copy link
Contributor

Choose a reason for hiding this comment

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

영속성 컨텍스트 이용하지 않고 JPQL 쓰신 이유가 있을까요? 궁금합니다.

Copy link
Collaborator

@mixedtape mixedtape left a comment

Choose a reason for hiding this comment

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

마지막까지 고생많으셨어요!

@EUNCHAEv1006 EUNCHAEv1006 merged commit e2313bc into dev Feb 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants