Skip to content

feat: smtp 설정 업데이트#84

Merged
kon28289 merged 1 commit intodevfrom
feat/mail
Apr 1, 2026
Merged

feat: smtp 설정 업데이트#84
kon28289 merged 1 commit intodevfrom
feat/mail

Conversation

@kon28289
Copy link
Copy Markdown
Contributor

@kon28289 kon28289 commented Apr 1, 2026

🚀 1. 개요

  • 네이버 로그인 2차인증이 잠시 비활성화되어 메일 전송이 되지 않던 문제가 발생했습니다.
  • 이에 2차인증을 다시 진행하고 smtp 관련 설정을 업데이트합니다.

📝 2. 주요 변경 사항

📸 3. 스크린샷 (API 테스트 결과)

Summary by CodeRabbit

릴리스 노트

  • 기타
    • 보안 관련 리소스가 업데이트되었습니다.

@kon28289 kon28289 requested a review from Juhye0k April 1, 2026 04:40
@kon28289 kon28289 self-assigned this Apr 1, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 1, 2026

Walkthrough

src/main/resources/security 서브모듈 포인터를 2a093de 커밋에서 d97e5b5 커밋으로 업데이트하였습니다. 이는 프로젝트가 참조하는 보안 리소스의 기저 컨텐츠 변경을 반영합니다.

Changes

Cohort / File(s) Summary
Submodule Update
src/main/resources/security
서브모듈 커밋 참조를 2a093de776b7409c0d9b1d366ec689d406290704에서 d97e5b5fdf3fab9a35a90f3c21c6d39f1fddd291로 업데이트

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 보안 자료가 새것으로 변신했네,
서브모듈이 쌩 하고 업데이트되니,
커밋 해시는 춤을 춘다네요,
조용하지만 중요한 변화,
보안 책임감 있게 나아가 봅시다! 🔐

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive PR 설명은 필수 템플릿 섹션(개요, 주요 변경 사항, 스크린샷)을 포함하고 있으나, 주요 변경 사항 섹션이 비어있어 구체적인 변경 내용이 부족합니다. 주요 변경 사항 섹션에 서브모듈 업데이트, SMTP 설정 변경 사항, 영향받는 파일 등 구체적인 변경 내용을 추가하여 완성하시기 바랍니다.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 SMTP 설정 업데이트를 명확히 언급하고 있으며, 실제 변경 내용인 보안 관련 서브모듈 업데이트와 관련이 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/mail

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/main/resources/security`:
- Line 1: PR 변경으로 src/main/resources/security 서브모듈 포인터가 바뀌었으니 대상 커밋에서 서브모듈이 가리키는
커밋을 확인하고 실제 파일 호환성을 검증하세요: src/main/resources/security 디렉터리가 초기화되어 있지 않으면 안전한
환경에서 git submodule init/update 후 src/main/resources/security 안에
application-database.yml, application-security.yml, application-mail.yml,
application-swagger.yml, application-wifi.yml, application-cloudinary.yml,
application-fcm.yml 파일이 모두 존재하는지 확인하고, 특히 application-mail.yml에 spring.mail 관련
키(예: host, port, username, password, auth, starttls 등)가 포함되어 있는지 체크하여 키 이름·구성
방식이 현재 dev 프로필에서 기대하는 구조와 일치하도록 수정하거나 문서화하세요.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c41ab821-e0d2-42d3-89ba-d550ae1f92cc

📥 Commits

Reviewing files that changed from the base of the PR and between 9f8da30 and 2765fef.

📒 Files selected for processing (1)
  • src/main/resources/security

Copy link
Copy Markdown
Contributor

@Juhye0k Juhye0k left a comment

Choose a reason for hiding this comment

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

LGTM!

@kon28289 kon28289 merged commit 3e3a0ac into dev Apr 1, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants