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.

Sign up for GitHub

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

Jump to bottom

회원가입 기능을 추가합니다 #25

Merged
merged 15 commits into from
Jul 29, 2023
Merged

회원가입 기능을 추가합니다 #25

merged 15 commits into from
Jul 29, 2023

Conversation

wisdom08
Copy link
Collaborator

작업 목적

  • 회원 가입 기능 구현

주요 변경점

  • 이전 초기 세팅 커밋에서 빼먹은 build.gradle 파일을 추가하고 회원가입 기능 구현까지 필요한 의존성을 추가했습니다.
  • POST /api/user 회원가입 API를 추가했습니다.
  • 패스워드 암호화는 jbcrypt 라이브러리를 사용해서 구현했습니다.

참고

  • 현재 회원가입 등록 테스트(UserServiceTest-signUp)의 경우, 아이디가 이미 등록된 경우에 실패하는 테스트입니다. 추후 testcontainer 도입을 통해 해결하면 어떨까 고민하고 있습니다.

@wisdom08 wisdom08 added the enhancement New feature or request label Jul 26, 2023
@wisdom08 wisdom08 added this to the 회원 관련 API milestone Jul 26, 2023
@wisdom08 wisdom08 self-assigned this Jul 26, 2023
ccommit-dev
ccommit-dev reviewed Jul 26, 2023
View reviewed changes
Copy link
Contributor

@ccommit-dev ccommit-dev left a comment

Choose a reason for hiding this comment

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

리뷰 내용 확인 부탁드립니다.

build.gradle Show resolved Hide resolved
src/main/java/com/bookbook/controller/UserController.java Outdated Show resolved Hide resolved
src/main/java/com/bookbook/controller/UserController.java Show resolved Hide resolved
src/main/java/com/bookbook/controller/UserController.java Show resolved Hide resolved
src/main/java/com/bookbook/service/UserService.java Show resolved Hide resolved
src/main/java/com/bookbook/dto/SignUpRequest.java Outdated Show resolved Hide resolved
src/main/java/com/bookbook/util/ResponseEntityConstants.java Show resolved Hide resolved
src/main/resources/mapper/userMapper.xml Outdated Show resolved Hide resolved
src/main/resources/mybatis-config.xml Outdated Show resolved Hide resolved
src/test/java/com/bookbook/service/UserServiceTest.java Show resolved Hide resolved
@wisdom08
docs: @SpringBootTest 어노테이션의 기능을 정리합니다
c727cae
@wisdom08
docs: @Mapper 어노테이션의 기능을 정리합니다
696f65a
@wisdom08
현재 필요하지 않은 파일을 삭제합니다
2ee2210
@wisdom08
데이터베이스 키워드를 테이블명, 칼럼값과의 구분을 위해 대문자로 변경하고 긴 경우에 키워드 기준으로 개행하도록 변경합니다
fca212a
@wisdom08
불필요한 SignUpRequest 객체 생성을 제거하고 encoder 처리를 SignUpRequest 객체 내부에서 …
91c6de8 
…하도록 변경합니다
@wisdom08
테스트가 데이터의 정합성 떄문에 실패하지 않도록 임시적으로 데이터를 초기화합니다
77cc841
@wisdom08
Reflection을 통해 동작하는 @RequestBody 에서 기본 생성자가 필요하기 때문에 추가해줍니다
33c85e2
@wisdom08
애플리케이션 전체에 적용할 공통 응답 코드를 추가합니다
70b4783
@wisdom08
익셉션 처리에 http status 코드를 추가합니다
183ae08
@wisdom08
SignupRequest 클래스로 옮겨 더이상 쓰지 않는 encoder를 제거합니다
fc2511d
@wisdom08
불필요한 코드 공백을 제거합니다
d238ac8
@wisdom08
공통 응답 코드를 적용하고 입력된 유저 데이터의 pk 값을 반환하게 변경합니다
ae75f2c
@wisdom08
Copy link
Collaborator Author

@ccommit-dev
안녕하세요 멘토님, 피드백 해주신 부분 반영하고, 궁금한 부분 코멘트로 질문드렸습니다.
확인부탁드립니다. 감사합니다.

@ccommit-dev
Copy link
Contributor

@wisdom08 질문주신 내용들 답변 드렸습니다. 해당 브랜치는 추가 질문들에 대한 답변을 해주신 후에 merge 해주시면 됩니다~

@wisdom08
api의 버전을 명시합니다
889cb3d
@wisdom08
비밀번호 암호화 로직을 기존대로 DTO에서 서비스 레이어로 변경합니다
92a6d1a 
- DTO는 data transfer object, 이름 그대로 transfer 목적이기 때문에 암호화로직이 있는 건
객체지향적이지 못함
@wisdom08 wisdom08 merged commit 8634832 into main Jul 29, 2023
@wisdom08 wisdom08 deleted the issues/#5-user-signup branch July 29, 2023 03:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ccommit-dev ccommit-dev ccommit-dev left review comments

Assignees

@wisdom08 wisdom08

Labels
enhancement New feature or request
Projects
None yet
Milestone
회원 관련 API
Development

Successfully merging this pull request may close these issues.
2 participants
@wisdom08 @ccommit-dev