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

SMTP를 통한 이메일 인증 구현 #133

Merged
merged 90 commits into from
Nov 9, 2023

Conversation

sososo0
Copy link
Contributor

@sososo0 sososo0 commented Nov 9, 2023

Summary

SMTP를 통한 이메일 인증 구현입니다.

Description

SMTP 서버 생성

  • 카카오 정책으로 인하여, 크램폴린 내에서는 smtp 통신이 되지 않는 다고 하여 SMTP 통신을 처리하는 서버를 하나 구축하였습니다.
  • 멀티클라우드 형식으로 구축하고 싶다는 생각이 들어, SMTP 통신을 처리하는 서버는 Naver Cloud 서버로 생성하였습니다.

Flask 코드 작성

  • 크램폴린과 SMTP 서버 사이의 SMTP 통신을 처리하기 위해 flask 코드를 작성하였습니다.
  • flask는 Dockerfile로 빌드되어, container 형태로 관리됩니다.
  • start.sh 파일을 이용하여 빌드된 Dockerfile로 flask container가 관리되게 하였습니다.
  • flask 관련 코드는 따로 flask 폴더를 만들어 그 안에 저장해두었습니다.

이메일 인증 코드 로직 수정

  • ec2상에서 실행되어야 하는 코드와 크램폴린 상에서 실행되어야 하는 코드가 달라, 해당 코드들을 함수로 분리시켜 작성하였습니다.
  • 따라서, 기존의 JavaMailSender 코드는 deploy 환경에서는 실행되지 않습니다.
  • 다른 이메일 인증이 필요한 부분도 같이 수정하였습니다.

+) 추가적인 질문 사항

제가, deploy 환경에서도 JavaMailSender의 Bean이 등록되어 있어야 해서 일단 우선적으로, deploy도 같이 넣어주었습니다. 이 부분은 어떻게 처리하는게 좋을까요?

해당 부분은 MailConfig에 있는 javaMailService 함수 입니다.

@Bean
    @Profile({"local", "product", "test", "deploy"})
    public JavaMailSender javaMailService() {
        JavaMailSenderImpl javaMailSender = new JavaMailSenderImpl();

@@ -39,4 +41,29 @@ private Properties getMailProperties() {
        properties.setProperty("mail.smtp.starttls.enable", "true");
        return properties;
    }

Related Issue

Issue Number: #128

sososo0 and others added 30 commits November 7, 2023 22:26
@sososo0 sososo0 added Priority: High 중요한 일 Type: Bug 문제 발견 Type: Maintenance refactoring 등 코드 개선 및 유지 보수 labels Nov 9, 2023
@sososo0 sososo0 self-assigned this Nov 9, 2023
@sososo0 sososo0 linked an issue Nov 9, 2023 that may be closed by this pull request
2 tasks
Copy link
Member

@jagaldol jagaldol left a comment

Choose a reason for hiding this comment

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

확인완료했습니다! 너무 고생하셨습니다..!🥹

@jagaldol jagaldol merged commit 37faf87 into Step3-kakao-tech-campus:weekly/10 Nov 9, 2023
1 check passed
@sososo0 sososo0 deleted the main branch November 9, 2023 17:30
@GoBeromsu
Copy link

우와,,, 멋있어요,,

@sososo0 sososo0 restored the main branch November 10, 2023 02:41
@sososo0 sososo0 deleted the main branch November 10, 2023 03:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: High 중요한 일 Type: Bug 문제 발견 Type: Maintenance refactoring 등 코드 개선 및 유지 보수
Projects
None yet
Development

Successfully merging this pull request may close these issues.

smtp 프록시 설정
3 participants