Skip to content
This repository was archived by the owner on Jan 28, 2024. It is now read-only.

로그인 토큰 검증 코드리뷰 내용 반영 #88

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

Conversation

lyle0101
Copy link
Member

@lyle0101 lyle0101 commented Feb 4, 2021

커밋 상세

  • 가변 변수이던 isValid 변수 대신 메소드의 리턴값을 사용하여 조건 검증
  • login token validator들을 TokenVerifier를 구현하도록 변경

설명

  • validator를 일단 구현하지 않도록 변경한 후, validator를 구현하지 않으므로 모듈의 이름도 변경했습니다.
  • 추후에 구글 등 다른 SNS 로그인 모듈이 생성될 수 있으므로 이전에 삭제했던 TokenVerifier 인터페이스를 다시 생성해 해당 인터페이스를 구현하도록 했습니다.

- 가변 변수이던 isValid 변수 대신 메소드의 리턴값을 사용하여 조건 검증
- login token validator들을 TokenVerifier를 상속하도록 변경
Copy link
Member

@junhaesung junhaesung left a comment

Choose a reason for hiding this comment

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

코멘트 추가했습니다.

추가로..
클래스 A 가 인터페이스 B 를 구현할때는 상속한다고 표현하지 않습니다. A 가 B 를 구현한다고 해야 맞는 표현입니다. (class A implements B ...)
한 인터페이스 C 가 다른 인터페이스 D 를 상속하는 경우에 C 가 D 를 상속한다고 표현할 수 있습니다. (interface C extends D)
spring data repository 를 사용할 때 JpaRepository<T, ID> 를 상속받는게 예시가 될 수 있겠네요

- 로그인 토큰이 존재하는 경우에만 검증을 진행하는 이유에 대한 주석 추가
- 통일성을 위해 isValid -> isVerified 로 메서드 이름 수정
@lyle0101 lyle0101 requested a review from junhaesung March 8, 2021 16:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants