davide/palladum-lightning
1
0
Fork 0
Code Issues Pull Requests Actions 5 Packages Projects Releases Wiki Activity

cln-plugin: don't panic if notification handler returns error

Browse Source 
Changelog-None
This commit is contained in:
daywalker90
2025-03-31 12:13:40 +02:00
committed by Rusty Russell
parent 87d862992a
commit 07e2a77be7
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
plugins/src/lib.rs
View File

@@ -813,7 +813,11 @@ where
match &self.wildcard_subscription {
Some(cb) => {
let call = cb(plugin.clone(), params.clone());
tokio::spawn(async move { call.await.unwrap() });
tokio::spawn(async move {
if let Err(e) = call.await {
log::warn!("Wildcard notification handler error: '{}'", e)
}
});
}
None => {}
};
@@ -823,7 +827,11 @@ where
match self.subscriptions.get(method) {
Some(cb) => {
let call = cb(plugin.clone(), params.clone());
tokio::spawn(async move { call.await.unwrap() });
tokio::spawn(async move {
if let Err(e) = call.await {
log::warn!("Notification handler error: '{}'", e)
}
});
}
None => {
if self.wildcard_subscription.is_none() {