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

로그인 처리 공통화를 위한 ArgumentResolver 및 Interceptor 구현 #156

Merged
merged 10 commits into from
Jul 31, 2024

Conversation

ksk0605
Copy link
Contributor

@ksk0605 ksk0605 commented Jul 31, 2024

PR의 목적이 무엇인가요?

로그인 처리 공통화를 위한 ArgumentResolver 및 Interceptor 구현

이슈 ID는 무엇인가요?

설명

  • LoginMemberArgumentResolver 구현 및 등록
  • AuthenticationCheckInterceptor 구현 및 등록
    • 로그인 API ("v1/auth/login") 제외
  • WebMvcConfig 구현

질문 혹은 공유 사항 (Optional)

@ksk0605 ksk0605 added BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현) labels Jul 31, 2024
@ksk0605 ksk0605 self-assigned this Jul 31, 2024
@ksk0605 ksk0605 changed the base branch from develop to develop-backend July 31, 2024 06:36
Copy link
Contributor

@pricelees pricelees left a comment

Choose a reason for hiding this comment

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

ㅎㅇ!

@@ -19,8 +19,7 @@ public class AuthController {
private final AuthService authService;

@PostMapping("/login")
public ResponseEntity<RestResponse<LoginResponse>> login(
@RequestBody LoginRequest loginRequest) {
public ResponseEntity<RestResponse<LoginResponse>> login(@RequestBody LoginRequest loginRequest) {
Copy link
Contributor

Choose a reason for hiding this comment

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

피드백 반영 감사요

@ksk0605 ksk0605 merged commit e772055 into develop-backend Jul 31, 2024
1 check passed
@Mingyum-Kim Mingyum-Kim deleted the feature/#154 branch July 31, 2024 06:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 관련 이슈입니다. 🌱 기능추가 feature (새로운 기능 구현)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

로그인 처리 공통화를 위한 ArgumentResolver 구현
4 participants