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 설정 smtp.properties 파일 분리 #26

Open
3 tasks done
gunny97h opened this issue Jan 13, 2025 · 1 comment
Open
3 tasks done

SMTP 설정 smtp.properties 파일 분리 #26

gunny97h opened this issue Jan 13, 2025 · 1 comment
Assignees
Labels
feat Spring, Mybatis, etc. ✔merge-ready Merged to develop but yet main

Comments

@gunny97h
Copy link
Contributor

gunny97h commented Jan 13, 2025

Task

  • application.properties에서 코드 분리
  • smtp.properties 구현
  • .gitignore에 smtp.properties 추가

Affected Files

- application.properties
- smtp.properties
- .gitignore
@gunny97h
Copy link
Contributor Author

gunny97h commented Jan 13, 2025

Description:

  • 현재 application.properties 파일에 포함된 SMTP 설정을 분리하여 smtp.properties 파일로 관리하려고 합니다.
  • 민감한 정보의 보안 강화와 환경별 설정을 용이하게 관리하기 위함입니다.

Tasks:

  1. src/main/resources/smtp.properties 파일 생성.

  2. smtp.properties에 SMTP 설정 추가:

spring.mail.host=smtp.gmail.com
spring.mail.port=587
spring.mail.username=
spring.mail.password=
spring.mail.properties.mail.smtp.auth=true
spring.mail.properties.mail.smtp.starttls.enable=true
spring.mail.properties.mail.smtp.connectiontimeout=5000
spring.mail.properties.mail.smtp.timeout=5000
spring.mail.properties.mail.smtp.writetimeout=5000
  1. application.properties에서 SMTP 관련 설정 제거 및 참조 설정 추가:
    spring.config.import=classpath:smtp.properties

  2. .gitignore에 smtp.properties 추가:
    /src/main/resources/smtp.properties

  3. 테스트를 통해 SMTP 설정이 정상적으로 동작하는지 확인.

Benefits:

  • 보안 강화: 민감한 정보를 소스 코드에서 분리.
  • 유지보수 용이: SMTP 설정 변경 시 별도의 파일만 수정.
  • 환경별 설정 관리 가능.

References:

@Queue-ri Queue-ri pinned this issue Jan 13, 2025
@C-H-Kim C-H-Kim unpinned this issue Jan 13, 2025
@C-H-Kim C-H-Kim pinned this issue Jan 13, 2025
@gunny97h gunny97h added the ✔merge-ready Merged to develop but yet main label Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Spring, Mybatis, etc. ✔merge-ready Merged to develop but yet main
Projects
None yet
Development

No branches or pull requests

1 participant