로그인 기능 리팩토링

src/main/java/kr/co/fastcampus/yanabada/domain/auth/service/AuthService.java

@@ -3,6 +3,7 @@
 import static kr.co.fastcampus.yanabada.domain.member.entity.ProviderType.EMAIL;
 import static kr.co.fastcampus.yanabada.domain.member.entity.RoleType.ROLE_USER;
 
+import java.util.Random;
 import kr.co.fastcampus.yanabada.common.exception.EmailDuplicatedException;
 import kr.co.fastcampus.yanabada.common.jwt.dto.TokenIssueResponse;
 import kr.co.fastcampus.yanabada.common.jwt.dto.TokenRefreshResponse;
@@ -43,14 +44,16 @@ public Long signUp(SignUpRequest signUpRequest) {
         }
 
         String encodedPassword = passwordEncoder.encode(signUpRequest.password());
+
         Member member = Member.builder()
-                .email(signUpRequest.email())
-                .nickName(signUpRequest.nickName())
-                .password(encodedPassword)
-                .phoneNumber(signUpRequest.phoneNumber())
-                .roleType(ROLE_USER)
-                .providerType(EMAIL)
-                .build();
+            .email(signUpRequest.email())
+            .nickName(signUpRequest.nickName())
+            .password(encodedPassword)
+            .phoneNumber(signUpRequest.phoneNumber())
+            .roleType(ROLE_USER)
+            .image(getRandomProfileImage())
+            .providerType(EMAIL)
+            .build();
 
         Member savedMember = memberRepository.save(member);
         return savedMember.getId();
@@ -67,13 +70,20 @@ public Long oauthSignUp(OauthSignUpRequest signUpRequest) {
             .password(encodedPassword)
             .phoneNumber(signUpRequest.phoneNumber())
             .roleType(ROLE_USER)
+            .image(getRandomProfileImage())
             .providerType(signUpRequest.provider())
             .build();
 
         Member savedMember = memberRepository.save(member);
         return savedMember.getId();
     }
 
+    private String getRandomProfileImage() {
+        Random random = new Random();
+        int randomNumber = random.nextInt(5) + 1;
+        return randomNumber + "profile.png";     //todo: 환경 변수 분리
+    }
+
     @Transactional
     public LoginResponse login(LoginRequest loginRequest) {
         UsernamePasswordAuthenticationToken authenticationToken = loginRequest.toAuthentication();

src/main/java/kr/co/fastcampus/yanabada/domain/chat/dto/SendChatMessage.java

@@ -27,7 +27,7 @@ public static SendChatMessage from(
             .chatRoomCode(chatRoom.getCode())
             .sendId(sender.getId())
             .senderNickname(sender.getNickName())
-            .senderProfileImage(sender.getImageUrl())
+            .senderProfileImage(sender.getImage())
             .content(content)
             .sendTime(sendTime)
             .build();

src/main/java/kr/co/fastcampus/yanabada/domain/chat/dto/response/ChatMessageInfoResponse.java

@@ -20,7 +20,7 @@ public static ChatMessageInfoResponse from(
     ) {
         return ChatMessageInfoResponse.builder()
             .senderId(sender.getId())
-            .senderImage(sender.getImageUrl())
+            .senderImage(sender.getImage())
             .senderNickname(sender.getNickName())
             .content(content)
             .sendDateTime(sendDateTime)

src/main/java/kr/co/fastcampus/yanabada/domain/chat/dto/response/ChatRoomSummaryResponse.java

@@ -29,7 +29,7 @@ public static ChatRoomSummaryResponse from(
     ) {
         return ChatRoomSummaryResponse.builder()
             .chatRoomCode(chatRoomCode)
-            .partnerImage(partner.getImageUrl())
+            .partnerImage(partner.getImage())
             .partnerNickname(partner.getNickName())
             .lastChatMessage(message.getContent())
             .lastSentMessageTime(message.getSendDateTime())

src/main/java/kr/co/fastcampus/yanabada/domain/member/controller/MemberController.java

@@ -4,7 +4,7 @@
 import kr.co.fastcampus.yanabada.common.response.ResponseBody;
 import kr.co.fastcampus.yanabada.common.security.PrincipalDetails;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.FcmTokenUpdateRequest;
-import kr.co.fastcampus.yanabada.domain.member.dto.request.ImgUrlModifyRequest;
+import kr.co.fastcampus.yanabada.domain.member.dto.request.ImageModifyRequest;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.NickNameModifyRequest;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.PasswordModifyRequest;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.PhoneNumberModifyRequest;
@@ -69,17 +69,6 @@ public ResponseBody<Void> modifyPhoneNumber(
         return ResponseBody.ok();
     }
 
-    @PutMapping("/image")
-    public ResponseBody<Void> modifyImage(
-        @AuthenticationPrincipal PrincipalDetails principalDetails,
-        @RequestBody @Valid ImgUrlModifyRequest imgUrlRequest
-    ) {
-        memberService.modifyImageUrl(
-            imgUrlRequest, principalDetails.email(), principalDetails.provider()
-        );
-        return ResponseBody.ok();
-    }
-
     @PutMapping("/fcm-token")
     public ResponseBody<Void> updateFcmToken(
         @AuthenticationPrincipal PrincipalDetails principalDetails,

src/main/java/kr/co/fastcampus/yanabada/domain/member/dto/request/ImgUrlModifyRequest.java → src/main/java/kr/co/fastcampus/yanabada/domain/member/dto/request/ImageModifyRequest.java

@@ -2,9 +2,9 @@
 
 import jakarta.validation.constraints.NotEmpty;
 
-public record ImgUrlModifyRequest(
+public record ImageModifyRequest(
     @NotEmpty(message = "패스워드가 비어있을 수 없습니다")
-    String imageUrl
+    String image
 ) {
 }
 

src/main/java/kr/co/fastcampus/yanabada/domain/member/dto/response/MemberDetailResponse.java

@@ -10,7 +10,7 @@ public record MemberDetailResponse(
     String email,
     String nickName,
     String phoneNumber,
-    String imageUrl,
+    String image,
     Integer point,
     ProviderType provider
 ) {
@@ -20,7 +20,7 @@ public static MemberDetailResponse from(Member member) {
             .email(member.getEmail())
             .nickName(member.getNickName())
             .phoneNumber(member.getPhoneNumber())
-            .imageUrl(member.getImageUrl())
+            .image(member.getImage())
             .point(member.getPoint())
             .provider(member.getProviderType())
             .build();

src/main/java/kr/co/fastcampus/yanabada/domain/member/dto/response/MemberSummaryResponse.java

@@ -7,14 +7,14 @@
 public record MemberSummaryResponse(
     Long id,
     String nickname,
-    String imageUrl
+    String image
 ) {
 
     public static MemberSummaryResponse from(Member member) {
         return MemberSummaryResponse.builder()
             .id(member.getId())
             .nickname(member.getNickName())
-            .imageUrl(member.getImageUrl())
+            .image(member.getImage())
             .build();
     }
 }

src/main/java/kr/co/fastcampus/yanabada/domain/member/entity/Member.java

@@ -27,7 +27,7 @@ public class Member extends BaseEntity {
     private String nickName;
     private String password;
     private String phoneNumber;
-    private String imageUrl;
+    private String image;
     @Builder.Default
     private Integer point = 0;
     @Enumerated(EnumType.STRING)
@@ -48,8 +48,8 @@ public void updatePhoneNumber(String phoneNumber) {
         this.phoneNumber = phoneNumber;
     }
 
-    public void updateImageUrl(String imageUrl) {
-        this.imageUrl = imageUrl;
+    public void updateImage(String image) {
+        this.image = image;
     }
 
     public void subtractPoint(int point) {

src/main/java/kr/co/fastcampus/yanabada/domain/member/service/MemberService.java

@@ -7,7 +7,7 @@
 import kr.co.fastcampus.yanabada.domain.auth.dto.response.EmailAuthResponse;
 import kr.co.fastcampus.yanabada.domain.auth.service.MailAuthService;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.FcmTokenUpdateRequest;
-import kr.co.fastcampus.yanabada.domain.member.dto.request.ImgUrlModifyRequest;
+import kr.co.fastcampus.yanabada.domain.member.dto.request.ImageModifyRequest;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.NickNameDuplCheckRequest;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.NickNameModifyRequest;
 import kr.co.fastcampus.yanabada.domain.member.dto.request.PasswordModifyRequest;
@@ -67,16 +67,6 @@ public void modifyPhoneNumber(
         member.updatePhoneNumber(phoneNumberRequest.phoneNumber());
     }
 
-    @Transactional
-    public void modifyImageUrl(
-        ImgUrlModifyRequest imgUrlRequest,
-        String email,
-        ProviderType providerType
-    ) {
-        Member member = memberRepository.getMember(email, providerType);
-        member.updateImageUrl(imgUrlRequest.imageUrl());
-    }
-
     @Transactional(readOnly = true)
     public EmailAuthResponse verifyEmail(
         EmailAuthRequest emailRequest