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

지원단위별 계정 통합 생성 시 이메일이 보내지지 않는 현상 해결 #376

Merged
merged 2 commits into from
Jun 15, 2024

Conversation

SongJaeHoonn
Copy link
Contributor

@SongJaeHoonn SongJaeHoonn commented Jun 14, 2024

Summary

#375

지원 단위 별로 합격자 계정 통합 생성시 이메일이 정상적으로 보내지지 않는 현상을 해결했습니다.

Tasks

  • EmailService에 있던 emailQueue를, 실제 사용하는 EmailAsyncService로 옮깁니다.
  • processEmileQueue에 스케줄러를 다시 도입합니다.

ETC

이전에 해당 부분을 리팩토링했을 때, 시스템 성능을 위해 스프링의 스케줄러를 삭제했습니다. 하지만 스케줄러를 삭제하니 EmailTask가 emailQueue에 저장되었다가 processEmailQueue 메소드에 의해 실제 이메일이 전송되는 과정에서, 비동기로 실행하기 때문에 다른 비동기 작업들이 완료되지 않아 첫 api 호출에서 이메일 큐의 내용을 처리하지 못하는 문제로 발생한 버그로 인지했습니다.
따라서 다시 스프링 스케줄러를 도입해 비동기 메소드를 주기적으로 호출해 큐에 남아있는 이메일들을 계속해서 전송하는 방식으로 변경했습니다.

Screenshot

여러 번 계정 삭제 및 반복으로 전송을 시도했을 때, 잘 전송이 되는 것을 볼 수 있습니다.
스크린샷 2024-06-15 132717

@SongJaeHoonn SongJaeHoonn requested a review from limehee as a code owner June 14, 2024 17:46
@SongJaeHoonn SongJaeHoonn requested a review from mingmingmon June 14, 2024 18:03
@SongJaeHoonn SongJaeHoonn self-assigned this Jun 14, 2024
@SongJaeHoonn SongJaeHoonn added the 🐞 Bug 버그 제보 및 수정 label Jun 14, 2024
Copy link
Collaborator

@mingmingmon mingmingmon left a comment

Choose a reason for hiding this comment

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

빠른 픽스 고생하셨습니다!!

@limehee
Copy link
Collaborator

limehee commented Jun 15, 2024

image
image

여전히 로컬 및 배포 환경에서 이메일이 전송되지 않고 있습니다. 확인 부탁드립니다.

@SongJaeHoonn
Copy link
Contributor Author

수정 완료했습니다.

@limehee
Copy link
Collaborator

limehee commented Jun 15, 2024

구체적으로 어떤 부분에서 왜 오류가 났는지 기록해주세요.

@SongJaeHoonn
Copy link
Contributor Author

버그 발생 이유 기록했습니다!

@limehee limehee merged commit 390493d into develop Jun 15, 2024
1 check passed
@limehee limehee deleted the fix/#375 branch June 15, 2024 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 Bug 버그 제보 및 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

지원 단위별 계정 통합 생성 시 이메일이 보내지지 않는 현상
3 participants