Skip to content

Commit

Permalink
Do not panic on attempting to force-close a subchannel
Browse files Browse the repository at this point in the history
We have run into this in production, it seems logging an error instead of
panicking is safest.
  • Loading branch information
klochowicz committed Sep 21, 2023
1 parent 695cec5 commit 6775c53
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion dlc-manager/src/manager.rs
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,10 @@ macro_rules! check_for_timed_out_channels {
let is_timed_out = timeout < $manager.time.unix_time_now();
if is_timed_out {
let sub_channel = if channel.is_sub_channel() {
unimplemented!();
error!("Force-closing subchannels is not supported; skipping closure.");
continue;

// TODO: Implement subchannel force closing
// let s = get_sub_channel_in_state!(
// $manager,
// channel.channel_id,
Expand Down

0 comments on commit 6775c53

Please sign in to comment.