Merge pull request #327 from mosu-dev/develop

feat: enhance user sign-up process with logging and phone number retr…

Author: polyglot-k

src/main/java/life/mosu/mosuserver/application/auth/SignUpService.java

@@ -6,10 +6,12 @@
 import life.mosu.mosuserver.presentation.auth.dto.Token;
 import life.mosu.mosuserver.presentation.auth.dto.request.SignUpAccountRequest;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.security.crypto.password.PasswordEncoder;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 
+@Slf4j
 @Service
 @RequiredArgsConstructor
 public class SignUpService {
@@ -27,6 +29,7 @@ public Token signUp(final SignUpAccountRequest request) {
 
     private UserJpaEntity doAccountStep(SignUpAccountRequest request) {
         UserJpaEntity user = request.toAuthEntity(passwordEncoder);
+        log.info("log : {}", user);
         return signUpAccountStepProcessor.process(user);
     }
 }

src/main/java/life/mosu/mosuserver/application/auth/processor/SignUpAccountStepProcessor.java

@@ -6,23 +6,26 @@
 import life.mosu.mosuserver.global.exception.ErrorCode;
 import life.mosu.mosuserver.global.processor.StepProcessor;
 import lombok.RequiredArgsConstructor;
+import lombok.extern.slf4j.Slf4j;
 import org.springframework.stereotype.Component;
 import org.springframework.transaction.annotation.Transactional;
 
 @Component
 @RequiredArgsConstructor
+@Slf4j
 public class SignUpAccountStepProcessor implements StepProcessor<UserJpaEntity, UserJpaEntity> {
 
     private final UserJpaRepository userRepository;
 
     @Transactional
     @Override
     public UserJpaEntity process(UserJpaEntity user) {
-        if (userRepository.existsByPhoneNumber(user.getPhoneNumber())) {
+        if (userRepository.existsByPhoneNumber(user.getOriginPhoneNumber())) {
             throw new CustomRuntimeException(ErrorCode.USER_ALREADY_EXISTS);
         } else if (userRepository.existsByLoginId(user.getLoginId())) {
             throw new CustomRuntimeException(ErrorCode.USER_ALREADY_EXISTS);
         }
+        log.info("Processing user sign-up: {}", user);
         return userRepository.save(user);
     }
 }

src/main/java/life/mosu/mosuserver/domain/user/entity/UserJpaEntity.java

@@ -125,4 +125,25 @@ public String getPhoneNumber() {
     public String getPhoneNumberWithoutHyphen() {
         return getPhoneNumber().replaceAll("-", "");
     }
+
+    public String getOriginPhoneNumber() {
+        return phoneNumber;
+    }
+
+    @Override
+    public String toString() {
+        return "UserJpaEntity{" +
+                "id=" + id +
+                ", loginId='" + loginId + '\'' +
+                ", password='" + password + '\'' +
+                ", gender=" + gender +
+                ", name='" + name + '\'' +
+                ", birth=" + birth +
+                ", phoneNumber='" + phoneNumber + '\'' +
+                ", customerKey='" + customerKey + '\'' +
+                ", agreedToMarketing=" + agreedToMarketing +
+                ", userRole=" + userRole +
+                ", provider=" + provider +
+                '}';
+    }
 }