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

[feat] 임시 로그인 기능 구현 #16

Merged
merged 11 commits into from
Jan 18, 2025

Conversation

dogsub
Copy link
Contributor

@dogsub dogsub commented Jan 12, 2025

개요

  • 로그인 관련 Dto(MemberRequestDto, MemberResponseDto, LoginRequestDto), Repository, Service, Controller 생성
  • 회원가입 API, 전체/단일 회원 정보 조회 API 구현
  • SecurityConfig 수정
  • global/security/jwt 패키지 생성 및 JwtProvider, JwtAuthenticationFilter 생성
  • JWT 인증을 통한 임시 로그인 기능 구현

PR 유형

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 주석 추가 및 수정
  • 문서 수정
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 파일 혹은 폴더명 수정
  • 파일 혹은 폴더 삭제

PR Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

📣 To Reviewers

일단 이번에 구현한 임시 로그인 기능은 사용자가 회원 가입 후 로그인 시 JWT 인증을 통해 Access Token을 발급해주는 기능으로 아래의 3가지 기능은 추후 추가할 예정입니다.

  • refresh token 추가
  • 예외 처리 추가
  • 회원 가입 시 Service 로직에서 PasswordEncoder 메서드 사용해서 DB에 저장 시 암호화

@dogsub dogsub added feat New feature or request 동동 labels Jan 12, 2025
@dogsub dogsub self-assigned this Jan 12, 2025
@dogsub dogsub changed the title [Feat] 임시 로그인 기능 구현 [feat] 임시 로그인 기능 구현 Jan 12, 2025
Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

임시 로그인 구현하시느라 고생하셨습니다! 임시 로그인이지만 해당 로직들에서 추후 소셜 로그인으로 파생될 수있다고 생각해서 수정했으면 하는 부분, 궁금한 부분, 논의했으면 하는 부분 조금 세세하게 남겼습니다! 리뷰가 좀 많아 시간은 걸릴 수 있지만 답변 부탁드리겠습니다!!!!

Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

추가 수정사항 리뷰 달았습니다!

Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

마지막 수정 요청사항입니다! 정말 고생하셨습니다!

Copy link
Contributor

@dyk-im dyk-im left a comment

Choose a reason for hiding this comment

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

리뷰가 길었는데 정말 고생하셨습니다! 다음 소셜로그인 파트 마저 진행 부탁드려요!

Copy link
Contributor

@KyunghwanChoi KyunghwanChoi left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

@dogsub dogsub merged commit f92dd8c into develop Jan 18, 2025
2 checks passed
@dogsub dogsub deleted the feat/#5-임시-로그인-기능-구현 branch January 18, 2025 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat New feature or request 동동
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[feat] 임시 로그인 기능 구현
4 participants