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

시큐리티 /access-token로 바뀐 url에 permitAll, 로그인 시 리프레시 토큰 무조건 새로 발급 #129

Merged
merged 4 commits into from
Nov 29, 2023

Conversation

jminkkk
Copy link
Collaborator

@jminkkk jminkkk commented Nov 29, 2023

PR 타입(하나 이상의 PR 타입을 선택해주세요)

  • 기능 추가
  • 기능 수정
  • 기능 삭제
  • 리팩토링
  • 버그 수정
  • 의존성, 환경 변수, 빌드 관련 코드 업데이트

작업 사항

  • 리프레시 토큰 발급 시 무조건 새로 발급
    • 기존 레거시 로직: 로그인 시 해당 회원의 만료되지 않은 리프레시 토큰 발급 있다면 새로 생성하지 않고 기존 것 리턴
    • 변경한 로직: 로그인 시 리프레시 토큰 무조건 새로 발급
  • 시큐리티 /access-token로 바뀐 url에 permitAll 부여
    • 필터에서 발생한 에러를 따로 처리하지 않아 예외 처리 X -> exceptionHandling 로 추가
  • MemberRepository 누락된 final 추가

관련 이슈

closes #124

@jminkkk jminkkk merged commit d49544a into dev Nov 29, 2023
1 check passed
@jminkkk jminkkk self-assigned this Oct 27, 2024
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.

엑세스 토큰 url 수정 및 리팩토링
1 participant