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

✨ Member 모듈 및 인증 AOP 구현 #14

Merged
merged 25 commits into from
Oct 13, 2024

Conversation

namkikim0718
Copy link
Member

#️⃣연관된 이슈

📝작업 내용

  • 회원 모듈을 생성했습니다.
  • 회원 모듈을 2개의 DB를 사용합니다. Member은 MySQL, MemberAdditionalInfo는 MongoDB를 사용합니다.
  • DB별 BaseEntity를 생성해 생성일시, 수정일시를 자동 저장합니다.
  • 회원의 초기 프로필 이미지는 깃허브 프로필 이미지를 사용하고 추후에 회원 기본정보 수정을 통해 다른 사진으로도 변경 가능합니다.
  • S3 관련 기능을 구현해 변경하면서 첨부한 이미지파일은 S3에 저장되고 UUID값으로 변경해 DB에 저장합니다.
  • 회원 관련 수정 API는 PutMapping을 활용했습니다. (PatchMapping으로 수정시 코드가 너무 난잡해져서 PutMapping을 사용했습니다.)
  • Common 모듈에서 인증 및 요청한 사용자의 memberId를 가져오기 위해 JWTUtil은 공통 모듈로 뺐습니다.
  • Common 모듈에서 AOP를 만들어 @AuthRequired 어노테이션으로 모든 모듈에서 인증 처리가 가능합니다.

💬리뷰 요구사항

�- 일단 회원 상세 정보 조회라는 API로 회원의 기본 정보와 추가정보를 한번에 조회하는 기능을 만들었는데, 필요할시 회원 기본 정보 조회 엔드포인트 및 추가정보 조회 엔드포인트를 따로 만들어야할 것 같습니다.

@namkikim0718 namkikim0718 self-assigned this Oct 10, 2024
@ghk01214 ghk01214 removed their request for review October 10, 2024 12:33
Copy link
Contributor

@yoonseopkim yoonseopkim left a comment

Choose a reason for hiding this comment

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

휴무일에도 대단하시네요... 고생하셨습니다!
당장 생각나는 거로는 추후에 커뮤니티 관련 기능에서 사용자끼리 서로 정보를 볼때 기본 정보만 볼 수 있게 분리해놓으면 좋을거 같긴한데 성능이 차이가 많이날지는 모르겠네요
근데 DB 가 서로 달라서 추후 관리측면을 생각한다면 엔드포인트를 분리하는것도 좋아보입니다 !

@namkikim0718
Copy link
Member Author

휴무일에도 대단하시네요... 고생하셨습니다! 당장 생각나는 거로는 추후에 커뮤니티 관련 기능에서 사용자끼리 서로 정보를 볼때 기본 정보만 볼 수 있게 분리해놓으면 좋을거 같긴한데 성능이 차이가 많이날지는 모르겠네요 근데 DB 가 서로 달라서 추후 관리측면을 생각한다면 엔드포인트를 분리하는것도 좋아보입니다 !

맞습니다! 다음주에 한번 커뮤니티 기능적인 부분들이랑 어떤 필드들을 넣어야할지 픽스하는 시간 한번 가지면 좋을 것 같습니다!

Copy link
Member

@speardragon speardragon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!!

@namkikim0718 namkikim0718 merged commit a5e7b19 into develop Oct 13, 2024
@namkikim0718 namkikim0718 deleted the 13-Member-모듈-및-인증-AOP-구현 branch October 29, 2024 05:02
@namkikim0718 namkikim0718 added ⚙ Setting 개발 환경 세팅 ✨ Feature 기능 개발 and removed enhancement labels Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 ⚙ Setting 개발 환경 세팅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BE | Spring] Member 모듈 및 인증 AOP 구현
3 participants