From 8ea3cd41b48cc96a599603c9d3e1a170870015fc Mon Sep 17 00:00:00 2001 From: Matt Corallo Date: Wed, 2 Oct 2024 18:23:31 +0000 Subject: [PATCH] Set the `dns_resolution` feature in `OMDomainResolver` `OMDomainResolver` actually does support building DNSSECProofs, so should be setting the `dns_resolution` `NodeFeature` flag. --- lightning-dns-resolver/Cargo.toml | 1 + lightning-dns-resolver/src/lib.rs | 8 ++++++++ 2 files changed, 9 insertions(+) diff --git a/lightning-dns-resolver/Cargo.toml b/lightning-dns-resolver/Cargo.toml index 979185636..46ea7e8ae 100644 --- 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 index f06b8abc3..cd8db4603 100644 --- 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()) } -- 2.39.5