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

JWT 리프레시 토큰 구현 #21

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

eunjee
Copy link
Contributor

@eunjee eunjee commented Jan 29, 2022

  • 회원가입, 로그인 서비스를 일부 수정했습니다.
  • DB에 JWT 테이블을 추가했습니다. (반영이 안될 수도 있기 때문에 db 레포지토리에 수정은 안했습니다!)

1. 액세스 토큰 갱신 API

  • 액세스 토큰 기간 만료 시 리프레시 토큰을 보낸다.
  • 리프레시 토큰이 만료되었는지 확인
  • 만료가 안되었다면 기존 리프레시 토큰과 새로운 액세스 토큰을 반환
  • 만료가 되었다면 새로운 리프레시 토큰과 액세스 토큰을 반환

2. 로그아웃 API

리프레시 토큰 사용 시 필요할 것 같아 만들었습니다.

  • 액세스 토큰으로 memberId 얻음
  • JWT 테이블의 리프레시 토큰을 null로 변경

저도 다시 생각해보니 굳이 필요는 없을 것 같은데 일단 PR만 올려두겠습니다!

@eunjee eunjee self-assigned this Jan 29, 2022
Copy link
Contributor

@kimtaehyun98 kimtaehyun98 left a comment

Choose a reason for hiding this comment

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

새롭게 접한 기술인텐데 빠르게 완성해주셨네요! 고생하셨습니다~ 👍👍
PR을 보니 해당 기술을 사용할지는 안드단에도 여쭤봐야될 것 같네요!

/*
새로운 액세스 토큰 혹은 리프레시 토큰이 필요한 경우
*/
@PostMapping("/accessToken")
Copy link
Contributor

Choose a reason for hiding this comment

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

안드단에서 리프레시 토큰을 요청하는 거군요! 😀

@@ -18,9 +18,8 @@
private String memberEmail;
private String memberPw;
private int memberSchoolId;
private int memberMajorId;
private String memberMajor;
Copy link
Contributor

Choose a reason for hiding this comment

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

이건 저번 PR에서 수정했던것 같은데 아직 머지가 안됐나보네요.. ㅠㅠ

this.jdbcTemplate = new JdbcTemplate(dataSource);
}

public void createRefreshToken(int memberId,String refreshToken){
Copy link
Contributor

Choose a reason for hiding this comment

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

DB에 JWT 테이블을 새로 생성한건가요??

@eunjee eunjee linked an issue Feb 3, 2022 that may be closed by this pull request
@kkkmj
Copy link
Contributor

kkkmj commented Feb 4, 2022

수고하셨습니다!!

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.

JWT 리프레시 토큰 구현
3 participants