Skip to content

Commit 1c525a2

Browse files
committed
fixup! Pass HRNResolver or DomainResolver into OnionMessenger
Use a more appropriate BuildError variant.
1 parent 48042a0 commit 1c525a2

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

bindings/ldk_node.udl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -398,6 +398,7 @@ enum BuildError {
398398
"NetworkMismatch",
399399
"AsyncPaymentsConfigMismatch",
400400
"DNSResolverSetupFailed",
401+
"PeerManagerSetupFailed,
401402
};
402403

403404
[Trait]

src/builder.rs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -194,6 +194,8 @@ pub enum BuildError {
194194
AsyncPaymentsConfigMismatch,
195195
/// An attempt to setup a DNS Resolver failed.
196196
DNSResolverSetupFailed,
197+
/// Failed to set up the peer manager.
198+
PeerManagerSetupFailed,
197199
}
198200

199201
impl fmt::Display for BuildError {
@@ -229,6 +231,9 @@ impl fmt::Display for BuildError {
229231
Self::DNSResolverSetupFailed => {
230232
write!(f, "An attempt to setup a DNS resolver has failed.")
231233
},
234+
Self::PeerManagerSetupFailed => {
235+
write!(f, "Failed to set up the peer manager.")
236+
},
232237
}
233238
}
234239
}
@@ -1757,7 +1762,7 @@ fn build_with_store_internal(
17571762
if let Ok(mut guard) = peer_manager_hook.lock() {
17581763
*guard = Some(Arc::downgrade(&peer_manager));
17591764
} else {
1760-
return Err(BuildError::DNSResolverSetupFailed);
1765+
return Err(BuildError::PeerManagerSetupFailed);
17611766
}
17621767

17631768
liquidity_source.as_ref().map(|l| l.set_peer_manager(Arc::downgrade(&peer_manager)));

0 commit comments

Comments
 (0)