Make cname config optional

WebThingsIO · Oct 22, 2024 · cc65a04 · cc65a04
1 parent df8c5ca
commit cc65a04
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 12 deletions.
diff --git a/src/config.rs b/src/config.rs
@@ -41,7 +41,7 @@ pub struct PdnsOptions {
     pub mx_records: Vec<(String,String)>,
     pub ns_records: Vec<Vec<String>>,
     pub txt_records: Vec<Vec<String>>,
-    pub cname_records: Vec<Vec<String>>,
+    pub cname_records: Option<Vec<Vec<String>>>,
     pub soa_record: String,
     pub www_addresses: Vec<String>,
     pub geoip: GeoIp,

diff --git a/src/pdns.rs b/src/pdns.rs
@@ -276,17 +276,19 @@ fn build_config_cname_response(
 ) -> Vec<PdnsLookupResponse> {
     let sanitized_qname = remove_trailing_dot(qname);
     let mut records = vec![];
-    for cname in &config.options.pdns.cname_records {
-        if cname[0] == sanitized_qname {
-            records.push(PdnsLookupResponse {
-                qtype: "CNAME".to_owned(),
-                qname: qname.to_owned(),
-                content: cname[1].to_owned(),
-                ttl: config.options.pdns.dns_ttl,
-                domain_id: None,
-                scope_mask: None,
-                auth: None,
-            });
+    if let Some(cname_records) = &config.options.pdns.cname_records {
+        for cname in cname_records {
+            if cname[0] == sanitized_qname {
+                records.push(PdnsLookupResponse {
+                    qtype: "CNAME".to_owned(),
+                    qname: qname.to_owned(),
+                    content: cname[1].to_owned(),
+                    ttl: config.options.pdns.dns_ttl,
+                    domain_id: None,
+                    scope_mask: None,
+                    auth: None,
+                });
+            }
         }
     }