Feature/4-find-user-id (#13)

* feat: 아이디 찾기 기능 구현 (#4) * Chore: 메일 송신자명 수정
Att-ies · Dec 29, 2022 · 89e14af · 89e14af
1 parent 09cc05d
commit 89e14af
Show file tree

Hide file tree

Showing 6 changed files with 67 additions and 66 deletions.
diff --git a/src/main/java/com/sptp/backend/common/exception/ErrorCode.java b/src/main/java/com/sptp/backend/common/exception/ErrorCode.java
@@ -18,6 +18,8 @@ public enum ErrorCode {
     EXIST_USER_ID(HttpStatus.BAD_REQUEST, "이미 존재하는 아이디입니다."),
     EXIST_USER_EMAIL(HttpStatus.BAD_REQUEST, "이미 존재하는 이메일입니다."),
     NOT_MATCH_PASSWORD(HttpStatus.BAD_REQUEST, "패스워드가 일치하지 않습니다."),
+    NOT_FOUND_EMAIL(HttpStatus.NOT_FOUND, "가입되지 않은 이메일 입니다."),
+    NOT_MATCH_USERNAME(HttpStatus.BAD_REQUEST, "잘못된 이름입니다."),
 
     //토큰 예외
     TOKEN_EXPIRED(HttpStatus.UNAUTHORIZED, "토큰이 만료되었습니다."),

diff --git a/src/main/java/com/sptp/backend/email/service/EmailService.java b/src/main/java/com/sptp/backend/email/service/EmailService.java
@@ -1,5 +1,5 @@
 package com.sptp.backend.email.service;
 
 public interface EmailService {
-    String sendMessage(String to)throws Exception;
+    void sendMessage(String to)throws Exception;
 }
diff --git a/src/main/java/com/sptp/backend/email/service/EmailServiceImpl.java b/src/main/java/com/sptp/backend/email/service/EmailServiceImpl.java
@@ -1,6 +1,8 @@
 package com.sptp.backend.email.service;
 
 import com.sptp.backend.email.service.EmailService;
+import com.sptp.backend.member.repository.Member;
+import com.sptp.backend.member.repository.MemberRepository;
 import lombok.RequiredArgsConstructor;
 import org.springframework.mail.MailException;
 import org.springframework.mail.javamail.JavaMailSender;
@@ -16,12 +18,11 @@
 public class EmailServiceImpl implements EmailService {
 
     private final JavaMailSender emailSender;
+    private final MemberRepository memberRepository;
 
-    public static final String ePw = createKey();
-
-    private MimeMessage createMessage(String to)throws Exception{
+    private MimeMessage createMessage(String to, String UserId)throws Exception{
         System.out.println("보내는 대상 : "+ to);
-        System.out.println("인증 번호 : "+ePw);
+        System.out.println("아이디 : "+ UserId);
         MimeMessage  message = emailSender.createMimeMessage();
 
         message.addRecipients(Message.RecipientType.TO, to);//보내는 대상
@@ -31,56 +32,27 @@ private MimeMessage createMessage(String to)throws Exception{
         msgg+= "<div style='margin:20px;'>";
         msgg+= "<h1> 안녕하세요 Atties입니다. </h1>";
         msgg+= "<br>";
-        msgg+= "<p>아래 코드를 복사해 입력해주세요<p>";
-        msgg+= "<br>";
-        msgg+= "<p>감사합니다.<p>";
-        msgg+= "<br>";
         msgg+= "<div align='center' style='border:1px solid black; font-family:verdana';>";
-        msgg+= "<h3 style='color:blue;'>회원가입 인증 코드입니다.</h3>";
+        msgg+= "<h3 style='color:blue;'>아래는 회원님의 아이디입니다.</h3>";
         msgg+= "<div style='font-size:130%'>";
-        msgg+= "CODE : <strong>";
-        msgg+= ePw+"</strong><div><br/> ";
+        msgg+= "회원ID : <strong>";
+        msgg+= UserId+"</strong><div><br/> ";
         msgg+= "</div>";
         message.setText(msgg, "utf-8", "html");//내용
-        message.setFrom(new InternetAddress("[email protected]","limjunho"));//보내는 사람
+        message.setFrom(new InternetAddress("[email protected]","Atties"));//보내는 사람
 
         return message;
     }
 
-    public static String createKey() {
-        StringBuffer key = new StringBuffer();
-        Random rnd = new Random();
-
-        for (int i = 0; i < 8; i++) { // 인증코드 8자리
-            int index = rnd.nextInt(3); // 0~2 까지 랜덤
-
-            switch (index) {
-                case 0:
-                    key.append((char) ((int) (rnd.nextInt(26)) + 97));
-                    //  a~z  (ex. 1+97=98 => (char)98 = 'b')
-                    break;
-                case 1:
-                    key.append((char) ((int) (rnd.nextInt(26)) + 65));
-                    //  A~Z
-                    break;
-                case 2:
-                    key.append((rnd.nextInt(10)));
-                    // 0~9
-                    break;
-            }
-        }
-        return key.toString();
-    }
     @Override
-    public String sendMessage(String to)throws Exception {
-        // TODO Auto-generated method stub
-        MimeMessage message = createMessage(to);
-        try{//예외처리
+    public void sendMessage(String to)throws Exception {
+        String UserId = memberRepository.findByEmail(to).get().getUserId();
+        MimeMessage message = createMessage(to, UserId);
+        try{
             emailSender.send(message);
         }catch(MailException es){
             es.printStackTrace();
             throw new IllegalArgumentException();
         }
-        return ePw;
     }
 }
diff --git a/src/main/java/com/sptp/backend/member/service/MemberService.java b/src/main/java/com/sptp/backend/member/service/MemberService.java
@@ -1,5 +1,6 @@
 package com.sptp.backend.member.service;
 
+import com.sptp.backend.member.web.dto.request.MemberFindIdRequestDto;
 import com.sptp.backend.member.web.dto.request.MemberLoginRequestDto;
 import com.sptp.backend.member.web.dto.request.MemberSaveRequestDto;
 import com.sptp.backend.member.repository.Member;
@@ -16,8 +17,11 @@
 import org.springframework.security.crypto.password.PasswordEncoder;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
+import org.springframework.util.CollectionUtils;
 
+import java.util.Arrays;
 import java.util.Collections;
+import java.util.List;
 import java.util.Optional;
 import java.util.concurrent.TimeUnit;
 
@@ -70,21 +74,25 @@ public TokenDto login(MemberLoginRequestDto dto) {
         return tokenDto;
     }
 
+    public Member findByEmail(MemberFindIdRequestDto dto) {
+
+        // 이메일 및 유저이름 유효성 체크
+        Member findMember = memberRepository.findByEmail(dto.getEmail())
+                .orElseThrow(() -> new CustomException(ErrorCode.NOT_FOUND_EMAIL));
+        if (!dto.getUsername().equals(findMember.getUsername())) {
+            throw new CustomException(ErrorCode.NOT_MATCH_USERNAME);
+        }
+
+        return findMember;
+    }
+
     public void logout(String accessToken) {
         Long expiration = jwtTokenProvider.getExpiration(accessToken);
 
         redisTemplate.opsForValue()
                 .set(accessToken, "blackList", expiration, TimeUnit.MILLISECONDS);
     }
 
-    public Optional<Member> findByEmail(String email) {
-        Optional<Member> findUser = memberRepository.findByEmail(email);
-        if (findUser.isPresent()) {
-            return findUser;
-        }
-        return null;
-    }
-
     public void checkDuplicateMemberID(String userId) {
         if (memberRepository.existsByUserId(userId)) {
             throw new CustomException(ErrorCode.EXIST_USER_ID);

diff --git a/src/main/java/com/sptp/backend/member/web/MemberController.java b/src/main/java/com/sptp/backend/member/web/MemberController.java
@@ -1,5 +1,6 @@
 package com.sptp.backend.member.web;
 
+import com.sptp.backend.member.web.dto.request.MemberFindIdRequestDto;
 import com.sptp.backend.jwt.service.JwtService;
 import com.sptp.backend.jwt.web.JwtTokenProvider;
 import com.sptp.backend.member.web.dto.request.MemberLoginRequestDto;
@@ -11,13 +12,18 @@
 import com.sptp.backend.email.service.EmailService;
 import com.sptp.backend.jwt.web.dto.TokenDto;
 import lombok.RequiredArgsConstructor;
+import org.springframework.http.HttpHeaders;
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.data.redis.core.RedisTemplate;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
+import org.springframework.ui.Model;
 import org.springframework.web.bind.annotation.*;
+import org.springframework.web.servlet.mvc.support.RedirectAttributes;
 
+import java.net.URI;
 import java.util.HashMap;
+import java.util.Optional;
 import java.util.Map;
 import java.util.concurrent.TimeUnit;
 
@@ -76,9 +82,9 @@ public ResponseEntity<?> refresh(@RequestHeader("refreshToken") String refreshTo
                 .build();
 
         return ResponseEntity.status(HttpStatus.OK).body(tokenResponseDto);
-
     }
 
+    // 로그아웃
     @PostMapping("/members/logout")
     public ResponseEntity logout(@RequestHeader("accessToken") String accessToken) {
 
@@ -87,24 +93,23 @@ public ResponseEntity logout(@RequestHeader("accessToken") String accessToken) {
         return new ResponseEntity(HttpStatus.OK);
     }
 
-    @PostMapping("/emailConfirm")
-    public String emailConfirm(@RequestParam String email) throws Exception {
+    // 아이디 찾기
+    @PostMapping("/members/id")
+    public ResponseEntity<?> findId(@RequestBody MemberFindIdRequestDto memberFindIdRequestDto) {
 
-        String confirm = emailService.sendMessage(email);
-        emailMap.put(email, confirm);
-        return confirm;
-    }
+        Member member = memberService.findByEmail(memberFindIdRequestDto);
 
-    @PostMapping("/emailConfirmCheck")
-    public String emailCheck(@RequestParam String email, @RequestParam String code) {
-        if (emailMap.get(email).equals(code)) {
-            emailMap.remove(email);
-            return "OK";
-        }else{
-            return "NO";
-        }
-    }
+        HttpHeaders headers = new HttpHeaders();
+        headers.setLocation(URI.create("/emailId?email=" + member.getEmail()));
 
+        return new ResponseEntity<>(headers, HttpStatus.MOVED_PERMANENTLY);
+    }
 
+    // 이메일로 아이디 발송
+    @PostMapping("/emailId")
+    public ResponseEntity emailConfirm(@RequestParam String email) throws Exception {
 
+        emailService.sendMessage(email);
+        return new ResponseEntity(HttpStatus.OK);
+    }
 }
diff --git a/src/main/java/com/sptp/backend/member/web/dto/request/MemberFindIdRequestDto.java b/src/main/java/com/sptp/backend/member/web/dto/request/MemberFindIdRequestDto.java
@@ -0,0 +1,14 @@
+package com.sptp.backend.member.web.dto.request;
+
+import lombok.*;
+
+@Getter
+@Setter
+@NoArgsConstructor
+@AllArgsConstructor
+@Builder
+public class MemberFindIdRequestDto {
+
+    private String username;
+    private String email;
+}