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

[UKET-7] 회원가입/로그인 모듈 마이그레이션 #108

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

0703kyj
Copy link
Contributor

@0703kyj 0703kyj commented Feb 23, 2025

📌 개요

  • 회원가입/로그인 모듈 마이그레이션

💻 작업사항

  • 회원가입/로그인 모듈 마이그레이션

❌ 주의사항

  • 1차적으로 모듈의 클래스들만 옮겨두었습니다.
  • 추후 로그인/회원가입 관련 컨트롤러 옮겨둘게용

💡 코드 리뷰 요청사항

@0703kyj 0703kyj self-assigned this Feb 23, 2025
Copy link
Contributor

@nonaninona nonaninona left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 이 PR 자체는 전부 다 옮긴 뒤에 머지하실 생각이신가여?

import com.uket.domain.auth.filter.JwtFilter;
import com.uket.app.auth.filter.JwtAccessDeniedHandler;
import com.uket.app.auth.filter.JwtAuthenticationEntryPoint;
import com.uket.app.auth.filter.JwtFilter;
import java.util.Collections;
import java.util.List;
import lombok.RequiredArgsConstructor;
Copy link
Contributor

Choose a reason for hiding this comment

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

admin의 SecurityConfig에 있던 "/admin/v1/auth/*"는 빠져도 되는 걸까요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

기존 API 유지하려면 필요하긴 한데 아마 회원가입 / 로그인 쪽 변경할거라 필요할 때 추가할게요!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants