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] Spring Secrity, JWT 의존성 추가 #9

Merged
merged 3 commits into from
Aug 23, 2024
Merged

Conversation

jw427
Copy link
Contributor

@jw427 jw427 commented Aug 22, 2024

Issue

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

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

반영 브랜치

feat/user_dependency -> dev

변경 사항

  • Spring Secrity, JWT 의존성 추가
  • config 패키지 및 파일 추가

@jw427 jw427 linked an issue Aug 23, 2024 that may be closed by this pull request
4 tasks
@jw427 jw427 added the feat label Aug 23, 2024
@ssunnykku ssunnykku self-requested a review August 23, 2024 01:48
Copy link
Contributor

@pie2457 pie2457 left a comment

Choose a reason for hiding this comment

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

리뷰 확인하시고 수정사항 반영해주시면 approve 하겠습니다 !
참고로 @Colum 어노테이션의 name은 테이블에 설정된 이름과 같다면 사용하지 않으셔도 괜찮습니다
카멜 케이스는 자동으로 스네이크 케이스로 변환시켜 줍니다😊

@Column(name = "code_id", nullable = false)
private Long codeId;

@OneToOne
Copy link
Contributor

Choose a reason for hiding this comment

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

@OneToOne 어노테이션은 왜 달았는지 궁금합니당 💭

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@OneToOne 어노테이션은 왜 달았는지 궁금합니당 💭

가입승인시 유저에게 발급되는 랜덤한 코드를 생성하는 api를 만들기 위한 엔티티이기 때문에, 회원과 1대1관계로 연결해주어야 한다고 생각했습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

그러면 연관관계의 주인이 User가 되는거죠?? User 테이블에 code라는 컬럼을 추가하는건 어떻게 생각하세요?? code라는 테이블이 따로 생겨야 하는 이유가 있을까요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

그러면 연관관계의 주인이 User가 되는거죠?? User 테이블에 code라는 컬럼을 추가하는건 어떻게 생각하세요?? code라는 테이블이 따로 생겨야 하는 이유가 있을까요??

가입이 승인된 후에는 필요하지 않은 코드이기도 하고, 만료시간을 두어서 일정 기간 안에 코드 등록을 하지 않으면 코드 재발급이 되게끔 하기 위해서는 따로 code 테이블을 두어서 관리하는게 낫지 않을까 생각했습니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

그럼 코드 테이블에 코드를 저장하는 경우는 어떤 경우가 있을까요??

Copy link
Contributor

Choose a reason for hiding this comment

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

그럼 코드 테이블에 코드를 저장하는 경우는 어떤 경우가 있을까요??

어제 강사님께서 사용자 관련으로 말씀해주신 것 바탕으로 저희가 서비스를 구성해봤습니다.
image

  1. 회원가입 후, 인증코드 승인에 상관없이 회원으로 등록
  2. 회원이 인증코드를 입력하면 회원등급 변경 (normal → premium)
  3. 인증과정에서 고객에게 더욱 풍부한 안내를 하기 위해 따로 테이블을 생성
  4. DB 관리를 위해 일정 시간이 지나면 인증코드는 자동으로 삭제할 예정

@Column(name = "refresh_token", nullable = false)
private String refreshToken;

@ManyToOne
Copy link
Contributor

Choose a reason for hiding this comment

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

@ManyToOne을 사용하실 때 즉시로딩과 지연로딩에 대해서 들어보신 적 있을까요!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@ManyToOne을 사용하실 때 즉시로딩과 지연로딩에 대해서 들어보신 적 있을까요!?

넵. @MayToOne 을 사용할 때 지연로딩이 권장된다고 배웠었는데, 코드 작성하면서 생각을 못했네요. 반영하겠습니다!


import java.util.UUID;

@Repository
Copy link
Contributor

Choose a reason for hiding this comment

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

@Repository 어노테이션은 사용하지 않으셔도 괜찮습니다!
이 글을 참고하시면 좋을 것 같아요~

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵 참고하겠습니다 감사합니다!

Copy link
Contributor

@pie2457 pie2457 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 👍🏼~~~

Copy link
Contributor

@K-0joo K-0joo left a comment

Choose a reason for hiding this comment

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

수고하셨습니다 ⭐⭐

@jw427 jw427 merged commit 58317aa into dev Aug 23, 2024
@jw427 jw427 deleted the feat/user_dependency branch August 23, 2024 05:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Spring Secrity, JWT 의존성 추가
5 participants