From d95345377b523b16cca35e4dba690cbbb79fcd48 Mon Sep 17 00:00:00 2001
From: Matthias Ahouansou <matthias@ahouansou.cz>
Date: Sat, 6 Jul 2024 17:31:31 +0100
Subject: [PATCH] fix: remove TLS name override when no SRV record is present
 (but properly)

The previous attempt only did so when no IP could be resolved, which isn't enough
---
 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 ca402f82..60ded056 100644
--- a/src/api/server_server.rs
+++ b/src/api/server_server.rs
@@ -583,6 +583,13 @@ async fn get_srv_destination(delegated_hostname: String) -> (FedDest, Option<Ins
             (add_port_to_hostname(&delegated_hostname), Some(timestamp))
         }
     } else {
+        // Removing in case there was previously a SRV record
+        services()
+            .globals
+            .tls_name_override
+            .write()
+            .unwrap()
+            .remove(&delegated_hostname);
         debug!("No SRV records found");
         (add_port_to_hostname(&delegated_hostname), None)
     }