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

Refactor #76 소셜 로그인을 위한 더미 패스워드 도입 #76

Merged
merged 1 commit into from
Nov 27, 2024

Conversation

hyxklee
Copy link
Member

@hyxklee hyxklee commented Nov 22, 2024

PR 내용

  • 기존 로그인을 사용하지 않기로 했으며, 소셜 로그인을 통한 인증시 password가 없다면 인증 오류가 발생하기 때문에 임시로 더미 패스워드 조치를 했습니다

PR 세부사항

  • 인증 객체 생성을 위한 userDetails 객체 생성시 password에 더미 패스워드를 전달하는 방식으로 임시 조치
  • 기존 로그인이 폐기됐지 때문에, JWT 필터를 소셜 로그인에 맞는, 즉 JWT에 의존하는 방식으로 수정할 예정
  • 로그인시, 그리고 매 요청시 userRepository에서 user를 조회해 role을 가져오고 있기 때문에, 이 방식은 지연이 크다고 생각해 차후 작업에서 수정할 계획입니다
  • -> role을 jwt에 담아서 확인하도록 하겠습니다

관련 스크린샷

X

주의사항

  • 현재 kakao social login redirect uri가 노션-정보-환경변수에 작성되어 구현되어 있으니, 서버 개발자 분들은 로그인 테스트 시 참고 바랍니다!

체크 리스트

  • 리뷰어 설정
  • Assignee 설정
  • Label 설정
  • 제목 양식 맞췄나요? (ex. #0 Feat: 기능 추가)
  • 변경 사항에 대한 테스트

@hyxklee hyxklee self-assigned this Nov 22, 2024
@hyxklee hyxklee changed the title Refactor #75 소셜 로그인을 위한 더미 패스워드 도입 Refactor #76 소셜 로그인을 위한 더미 패스워드 도입 Nov 22, 2024

@RequiredArgsConstructor
@Slf4j
public class JwtAuthenticationProcessingFilter extends OncePerRequestFilter {

private static final String NO_CHECK_URL = "/api/v1/login";
private final String DUMMY = "DUMMY_PASSWORD";
Copy link
Member

Choose a reason for hiding this comment

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

임시로 수정된 내용 확인했습니다 !

Copy link
Collaborator

@jj0526 jj0526 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

@hyxklee hyxklee merged commit c4fb6fa into develop Nov 27, 2024
2 checks passed
@hyxklee hyxklee deleted the Refactor/#74/jwt-필터-수정 branch November 27, 2024 04:35
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.

3 participants