MirrorHub/conduit
1
0
Fork 0
mirror of https://gitlab.com/famedly/conduit.git synced 2024-12-29 12:54:36 +01:00
Code Issues Releases Activity

fix: remove TLS name override when no SRV record is present

Browse source 
this could have been an issue in cases where there was previously a SRV record, but later got removed
This commit is contained in:
Matthias Ahouansou 2024-07-06 17:06:11 +01:00
parent 1f313c6807
commit 11187b3fad
No known key found for this signature in database
1 changed files with 7 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/api/server_server.rs
View file

@ -564,6 +564,13 @@ async fn get_srv_destination(delegated_hostname: String) -> (FedDest, Option<Ins
(override_ip.iter().collect(), force_port.unwrap_or(8448)), (override_ip.iter().collect(), force_port.unwrap_or(8448)),
); );
} else { } 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"); warn!("Using SRV record, but could not resolve to IP");
} }