From 11187b3fad47dca8677334e733370f19309a52e1 Mon Sep 17 00:00:00 2001
From: Matthias Ahouansou <matthias@ahouansou.cz>
Date: Sat, 6 Jul 2024 17:06:11 +0100
Subject: [PATCH] fix: remove TLS name override when no SRV record is present

this could have been an issue in cases where there was previously a SRV record, but later got removed
---
 src/api/server_server.rs | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/src/api/server_server.rs b/src/api/server_server.rs
index 6fd131c5..ca402f82 100644
--- a/src/api/server_server.rs
+++ b/src/api/server_server.rs
@@ -564,6 +564,13 @@ async fn get_srv_destination(delegated_hostname: String) -> (FedDest, Option<Ins
                     (override_ip.iter().collect(), force_port.unwrap_or(8448)),
                 );
         } else {
+            // Removing in case there was previously a SRV record
+            services()
+                .globals
+                .tls_name_override
+                .write()
+                .unwrap()
+                .remove(&delegated_hostname);
             warn!("Using SRV record, but could not resolve to IP");
         }