-
Notifications
You must be signed in to change notification settings - Fork 1
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
[BE] jwt principal 추가 #111
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다~!
@@ -15,6 +17,8 @@ | |||
@RequiredArgsConstructor | |||
@Service | |||
public class AuthService { | |||
private static final String USER_ID = "userId"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
클래스와 필드 사이는 한 줄 띄어주세용~
public class AuthService {
private static final String USER_ID = "userId";
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class Principal { | ||
private static final String USER_ID = "userId"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
여기도 한 줄 띄어주세요~!
@Builder | ||
@AllArgsConstructor | ||
@NoArgsConstructor | ||
public class Principal { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
정적 팩터리 메서드를 생성했는데 @AllArgsConstructor
, @NoArgsConstructor
를 달아주신 이유가 있나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 수정하겠습니다! 감사합니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
고생하셨습니다 👍🏼👍🏼👍🏼
@@ -15,6 +17,8 @@ | |||
@RequiredArgsConstructor | |||
@Service | |||
public class AuthService { | |||
private static final String USER_ID = "userId"; | |||
private static final String LOGIN_ID = "loginId"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요기 상수 처리는 앞으로를 위한건가요!? 지금은 한군데씩 밖에 사용을 안하는거죠?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
넵! 문자열은 보통 상수 처리하는 편입니다
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
문자열이랑 상수 이름이 같아서 제거했습니다!
Issues
What is this PR? 👓
Key changes 🔑
To reviewers 👋