From 3306fadc472368685f44d2262cc78eb8a3633194 Mon Sep 17 00:00:00 2001
From: the-djmaze <>
Date: Tue, 27 Aug 2024 13:19:30 +0200
Subject: [PATCH] MS autodiscover priorities DNS over subdomain

---
 .../RainLoop/Providers/Domain/Autoconfig.php  | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/snappymail/v/0.0.0/app/libraries/RainLoop/Providers/Domain/Autoconfig.php b/snappymail/v/0.0.0/app/libraries/RainLoop/Providers/Domain/Autoconfig.php
index 75d94a651..b104276d8 100644
--- a/snappymail/v/0.0.0/app/libraries/RainLoop/Providers/Domain/Autoconfig.php
+++ b/snappymail/v/0.0.0/app/libraries/RainLoop/Providers/Domain/Autoconfig.php
@@ -168,16 +168,6 @@ abstract class Autoconfig
 	 */
 	private static function autodiscover(string $domain) : ?array
 	{
-		foreach ([
-			"https://{$domain}",
-			"https://autodiscover.{$domain}",
-			"http://autodiscover.{$domain}"
-		] as $host) {
-			$result = static::autodiscover_resolve($host, $domain);
-			if ($result) {
-				return $result;
-			}
-		}
 		foreach (\SnappyMail\DNS::SRV("_autodiscover._tcp.{$domain}") as $record) {
 			if (443 == $record['port']) {
 				$result = static::autodiscover_resolve("https://{$record['target']}", $domain);
@@ -190,6 +180,16 @@ abstract class Autoconfig
 				return $result;
 			}
 		}
+		foreach ([
+			"https://{$domain}",
+			"https://autodiscover.{$domain}",
+			"http://autodiscover.{$domain}"
+		] as $host) {
+			$result = static::autodiscover_resolve($host, $domain);
+			if ($result) {
+				return $result;
+			}
+		}
 		return null;
 	}