From: Mark Felder <feld@feld.me>
Date: Sat, 6 Feb 2021 17:55:25 +0000 (-0600)
Subject: Remove No MX lookups setting
X-Git-Url: https://git.squeep.com/?a=commitdiff_plain;h=6ffe15cc9feadecf5e6756cb3db3240fa9eb63c2;p=akkoma

Remove No MX lookups setting

This setting defaults to false so the relay host will be used in an MX query so
multiple SMTP servers can be used. gen_smtp code states that all records returned from the
MX query are attempted in order and only a permanent SMTP error will stop the client from
attempting other servers in the list. Connection failures, TLS issues, etc will cause it to
try the next host.

If there is no MX record associated with the relay host, it automatically tries connecting to it
directly.

There is really no reason to expose this to end users. The default value is fine for everyone.
---

diff --git a/config/description.exs b/config/description.exs
index 85f90ca3e..8922a3626 100644
--- a/config/description.exs
+++ b/config/description.exs
@@ -295,13 +295,6 @@ config :pleroma, :config_description, [
         description: "`Swoosh.Adapters.SMTP` adapter specific setting",
         suggestions: [5]
       },
-      %{
-        group: {:subgroup, Swoosh.Adapters.SMTP},
-        key: :no_mx_lookups,
-        label: "No MX lookups",
-        type: :boolean,
-        description: "`Swoosh.Adapters.SMTP` adapter specific setting"
-      },
       %{
         group: {:subgroup, Swoosh.Adapters.Sendgrid},
         key: :api_key,