Set the dns_resolution feature in OMDomainResolver

`OMDomainResolver` actually does support building DNSSECProofs, so should be setting the `dns_resolution` `NodeFeature` flag.
TheBlueMatt · Oct 29, 2024 · c1dfc8c · c1dfc8c
1 parent 8ffeef1
commit c1dfc8c
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 0 deletions.
diff --git a/lightning-dns-resolver/Cargo.toml b/lightning-dns-resolver/Cargo.toml
@@ -5,6 +5,7 @@ edition = "2021"
 
 [dependencies]
 lightning = { version = "0.0.124-rc1", path = "../lightning", default-features = false }
+lightning-types = { version = "0.1", path = "../lightning-types", default-features = false }
 dnssec-prover = { version = "0.6", default-features = false, features = [ "std", "tokio" ] }
 tokio = { version = "1.0", default-features = false, features = ["rt"] }
 

diff --git a/lightning-dns-resolver/src/lib.rs b/lightning-dns-resolver/src/lib.rs
@@ -21,6 +21,8 @@ use lightning::onion_message::messenger::{
 	MessageSendInstructions, Responder, ResponseInstruction,
 };
 
+use lightning_types::features::NodeFeatures;
+
 #[cfg(not(any(target_pointer_width = "32", target_pointer_width = "64")))]
 const WE_REQUIRE_32_OR_64_BIT_USIZE: u8 = 424242;
 
@@ -107,6 +109,12 @@ where
 		None
 	}
 
+	fn provided_node_features(&self) -> NodeFeatures {
+		let mut features = NodeFeatures::empty();
+		features.set_dns_resolution_optional();
+		features
+	}
+
 	fn release_pending_messages(&self) -> Vec<(DNSResolverMessage, MessageSendInstructions)> {
 		core::mem::take(&mut *self.state.pending_replies.lock().unwrap())
 	}