Skip to content

Comments

[メール] MS365認証時のReply-To設定を追加しました#2334

Merged
masaton0216 merged 4 commits intomasterfrom
OW-2703
Jan 6, 2026
Merged

[メール] MS365認証時のReply-To設定を追加しました#2334
masaton0216 merged 4 commits intomasterfrom
OW-2703

Conversation

@masaton0216
Copy link
Contributor

@masaton0216 masaton0216 commented Jan 6, 2026

概要

MS365認証(Microsoft Graph Transport)を使用したメール送信時に、Reply-To(返信先)アドレスを適切に設定する機能を追加しました。

変更の背景・目的

MS365認証でメール送信を行う際、OAuth2設定の送信者アドレスと異なるFromアドレスを使用する場合、受信者が返信した際に適切なアドレスに返信されるようにする必要があります。この変更により、FromアドレスがOAuth2設定と異なる場合、自動的にReply-Toヘッダーが設定されるようになります。

主な変更内容

機能追加

  • MicrosoftGraphTransport::convertToGraphMessage() メソッドにReply-To設定ロジックを追加
    • 明示的にReply-Toが設定されている場合:そのまま使用
    • FromアドレスがOAuth2送信者アドレスと異なる場合:FromアドレスをReply-Toに自動設定
    • FromアドレスがOAuth2送信者アドレスと同じ場合:Reply-Toを設定しない

テスト追加

以下の3つのテストケースを追加し、動作を保証しました:

  1. FromがOAuth2設定アドレスと異なる場合にReply-Toが設定されることの確認
  2. FromがOAuth2設定アドレスと同じ場合にReply-Toが設定されないことの確認
  3. 明示的なReply-Toが優先されることの確認

レビュー指摘対応

以下の改善を実施しました:

  1. コメントの簡潔化: Reply-To設定部分のコメントを2行から1行に統合
  2. エッジケース処理の追加: Fromアドレスがnullまたは空配列の場合の安全な処理を追加
  3. テストケース追加: エッジケースに対応する2つのテストケースを追加
    • testReplyToNotSetWhenFromIsNull(): Fromがnullの場合
    • testReplyToNotSetWhenFromIsEmpty(): Fromが空配列の場合

レビュー完了希望日

特になし

関連PR / Issues

なし

参考情報

なし

DB変更

  • あり
  • なし

チェックリスト

@masaton0216 masaton0216 self-assigned this Jan 6, 2026
@masaton0216 masaton0216 added the enhancement 機能強化 label Jan 6, 2026
@masaton0216 masaton0216 merged commit f79ffc3 into master Jan 6, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement 機能強化

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant