Conversation
net/smtpでGmailに直接送信していたパスワードリセットメールをResend経由に置き換え。 drivers/mailにMailer interfaceとResend実装を切り出し、wireでDIするようにした。 RESEND_API_KEYの設定が必要(NUTMEG_MAIL_PASSWORDは不要になった)。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SRNg88ERdSubEcCQh2YcQ7
2 tasks
このPRの変更とは無関係だが、air@latestがv1.67.4に上がりgo1.26.0を 要求するようになったため、golang:1.25.5-alpineベースのapiイメージの docker buildがE2E CIで失敗していた。go1.25系で動くv1.67.1に固定する。 同じ修正を develop 向けの #1120 としても切り出し済み。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SRNg88ERdSubEcCQh2YcQ7
Deploying finansu with
|
| Latest commit: |
0d6f352
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://e1e9303b.finansu.pages.dev |
| Branch Preview URL: | https://claude-email-local-test-impl.finansu.pages.dev |
Collaborator
Author
|
Generated by Claude Code |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
対応Issue
resolve #0
概要
パスワードリセットメールの送信方法を、
net/smtpによるGmail直接送信から Resend 経由に変更しました。api/drivers/mailにMailer(Client)interfaceとResend実装を新規作成api/externals/repository/password_reset_tokens.goのSendResetEmailを、Resend Client経由で送信するように変更api/internals/di/wire.goにProvideMailClientを追加し、wire_gen.goを再生成してDIに組み込みnet/smtp/NUTMEG_MAIL_PASSWORDは不要になったため削除(NUTMEG_MAIL_SENDERは送信元アドレスとして引き続き使用)新たに環境変数
RESEND_API_KEYの設定が必要です。画面スクリーンショット等
テスト項目
RESEND_API_KEYを設定した状態で/password_reset/requestを叩き、パスワードリセットメールが届くことgo build ./...が通ること(確認済み)備考
mail.Clientをモックに差し替え可能です。delivered@resend.devなど)宛に送信することでドメイン未検証でも送信結果を確認できます。🤖 Generated with Claude Code
https://claude.ai/code/session_01SRNg88ERdSubEcCQh2YcQ7
Generated by Claude Code