Skip to content

Commit 6ccfce8

Browse files
authored
Merge pull request #205 from DevKor-github/feat/mail
Fix: 예외 오류 수정
2 parents d47ee2e + a55bc03 commit 6ccfce8

2 files changed

Lines changed: 14 additions & 8 deletions

File tree

src/main/java/devkor/com/teamcback/domain/suggestion/controller/SuggestionController.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,7 @@ public CommonResponse<CreateSuggestionRes> createSuggestion(
4444
@AuthenticationPrincipal UserDetailsImpl userDetail,
4545
@Parameter(description = "건의 제목, 분류, 내용, 이메일", required = true)
4646
@RequestPart(value = "req") CreateSuggestionReq req,
47-
@Parameter(description = "건의 사진") @RequestPart(value = "images", required = false) List<MultipartFile> images
48-
) throws MessagingException {
47+
@Parameter(description = "건의 사진") @RequestPart(value = "images", required = false) List<MultipartFile> images) {
4948
Long userId = userDetail == null ? null : userDetail.getUser().getUserId();
5049
return CommonResponse.success(suggestionService.createSuggestion(userId, req, images));
5150
}

src/main/java/devkor/com/teamcback/domain/suggestion/service/SuggestionService.java

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@
2020
import devkor.com.teamcback.infra.s3.S3Util;
2121
import jakarta.mail.MessagingException;
2222
import lombok.RequiredArgsConstructor;
23+
import lombok.extern.slf4j.Slf4j;
2324
import org.springframework.data.domain.Page;
2425
import org.springframework.data.domain.PageRequest;
2526
import org.springframework.data.domain.Pageable;
@@ -33,6 +34,7 @@
3334
import java.util.ArrayList;
3435
import java.util.List;
3536

37+
@Slf4j
3638
@Service
3739
@RequiredArgsConstructor
3840
public class SuggestionService {
@@ -46,7 +48,7 @@ public class SuggestionService {
4648
*/
4749
@Transactional
4850
@UpdateScore(addScore = 3)
49-
public CreateSuggestionRes createSuggestion(Long userId, CreateSuggestionReq req, List<MultipartFile> images) throws MessagingException {
51+
public CreateSuggestionRes createSuggestion(Long userId, CreateSuggestionReq req, List<MultipartFile> images) {
5052
User user = null;
5153
if(userId != null) user = findUser(userId);
5254
Suggestion suggestion = new Suggestion(req, user);
@@ -64,11 +66,16 @@ public CreateSuggestionRes createSuggestion(Long userId, CreateSuggestionReq req
6466
Suggestion savedSuggestion = suggestionRepository.save(suggestion);
6567

6668
// 이메일 전송
67-
emailService.sendNotificationMessage(savedSuggestion.getTitle(),
68-
user == null ? "익명" : user.getUsername(),
69-
savedSuggestion.getSuggestionType().getType(),
70-
savedSuggestion.getContent(),
71-
savedSuggestion.getImages());
69+
try {
70+
// 이메일 전송
71+
emailService.sendNotificationMessage(savedSuggestion.getTitle(),
72+
user == null ? "익명" : user.getUsername(),
73+
savedSuggestion.getSuggestionType().getType(),
74+
savedSuggestion.getContent(),
75+
savedSuggestion.getImages());
76+
} catch (MessagingException e) {
77+
log.error("이메일 전송 실패: {}", e.getMessage());
78+
}
7279

7380
return new CreateSuggestionRes(savedSuggestion);
7481
}

0 commit comments

Comments
 (0)