smtp_should_reject と discourse-smtp-fast-rejection を削除

開発
1

Seeing smtp_should_reject “400 bad request” failures on bounce processingからの議論を継続します。

ここでは、私たちが内部で行った議論をまとめるためのトピックを作成します。

smtp_should_reject のコードパスが正しく機能しておらず、本来拒否されるべきではない状況でメールが拒否されていることを特定しました。

具体的には、以下のケースです。

  • バウンスメッセージ（envelope-from が null のもの）
  • 転送されたメールの不一致（高速拒否チェックでは envelope-to が使用されていたのに対し、Discourse は To/CC ヘッダーを使用している）

これにより、セルフホストユーザーに以下の問題が発生しています。

このコードを維持する意向はないため、完全に削除することを検討しています。

Discourse PR:

mail-receiver PR:

「いいね！」 4
2

これら2つの修正は、互いに独立して機能する（つまり、どちらか一方を適用すれば問題が解決する）ように見え、どちらもセルフホストしているユーザーが通常通り更新する以上の労力を必要としないように思えますが、それで正しいですか？

3

これは正しいです。私の動機は、人々がDiscourseほど頻繁にmail-receiverを更新しない可能性があると推測しているため、それらの人々がまだ修正を受け取れるようにしたいということです。

「いいね！」 2