-
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
feat: 로그인 여부 확인 API #913
feat: 로그인 여부 확인 API #913
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.
늦은 밤까지 고생하셨습니다.
몇가지만 체크해주세요!
@Operation(summary = "로그인 여부 확인") | ||
@SecurityRequirement(name = "Jwt Authentication") | ||
@GetMapping("/user/check/login") | ||
ResponseEntity<Void> checklogin( |
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.
밑에보니 checkLogin인데 통일하는 것이 좋을것 같습니다.
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 record UserAccessTokenRequest( | ||
@Schema(description = "access_token", example = "eyJ0eXAiOiJKV1QiLCJhbGcIUzUxMic9.eyJpZCI6NTM5NhwIjoxkzI3MTA5ODE5fQ.rLEYGQfKI5_24ZlwLVwlgwnriqySPKwXNOeTRrbmxoCtlOzCVvM8FFcO9BA2vkqsmhf-w", requiredMode = REQUIRED) | ||
@NotNull(message = "access_token을 입력해주세요.") |
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.
@NotBlank
를 안쓰신 이유가 궁금합니다!
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.
@NotBlank
있으면 @NotNull
은 없어도 될 거 같아요
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.
아...맞다..ㅋㅋ 감사합니다
@@ -171,6 +155,15 @@ public ResponseEntity<Void> findPassword( | |||
return new ResponseEntity<>(HttpStatusCode.valueOf(201)); | |||
} | |||
|
|||
@GetMapping("/user/check/login") | |||
public ResponseEntity<Void> checklogin( |
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.
이곳도 동일하게 checkLogin 으로 바꿔주시면 좋겠습니다!
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 record UserAccessTokenRequest( | ||
@Schema(description = "access_token", example = "eyJ0eXAiOiJKV1QiLCJhbGcIUzUxMic9.eyJpZCI6NTM5NhwIjoxkzI3MTA5ODE5fQ.rLEYGQfKI5_24ZlwLVwlgwnriqySPKwXNOeTRrbmxoCtlOzCVvM8FFcO9BA2vkqsmhf-w", requiredMode = REQUIRED) | ||
@NotNull(message = "access_token을 입력해주세요.") |
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.
@NotBlank
있으면 @NotNull
은 없어도 될 거 같아요
@@ -18,24 +20,6 @@ | |||
import org.springframework.web.bind.annotation.RestController; | |||
import org.springframework.web.servlet.ModelAndView; | |||
|
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.
A
안 쓰는 import 정리 굿
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항