Skip to content

Commit c1894f9

Browse files
committed
Do not rely on assertions being enabled in ChannelManagerConstructor
1 parent 419e5b5 commit c1894f9

File tree

1 file changed

+10
-2
lines changed

1 file changed

+10
-2
lines changed

src/main/java/org/ldk/batteries/ChannelManagerConstructor.java

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,11 @@ public ChannelManagerConstructor(byte[] channel_manager_serialized, byte[][] cha
8888
this.peer_manager = PeerManager.of(channel_manager.as_ChannelMessageHandler(), (IgnoringMessageHandler.of()).as_RoutingMessageHandler(), keys_interface.get_node_secret(), random_data, logger);
8989
}
9090
NioPeerHandler nio_peer_handler = null;
91-
try { nio_peer_handler = new NioPeerHandler(this.peer_manager); } catch (IOException e) { assert false; }
91+
try {
92+
nio_peer_handler = new NioPeerHandler(this.peer_manager);
93+
} catch (IOException e) {
94+
throw new IllegalStateException("We should never fail to construct nio objects unless we're on a platform that cannot run LDK.");
95+
}
9296
this.nio_peer_handler = nio_peer_handler;
9397
if (filter != null) {
9498
for (ChannelMonitor monitor : monitors) {
@@ -118,7 +122,11 @@ public ChannelManagerConstructor(Network network, UserConfig config, byte[] curr
118122
this.peer_manager = PeerManager.of(channel_manager.as_ChannelMessageHandler(), (IgnoringMessageHandler.of()).as_RoutingMessageHandler(), keys_interface.get_node_secret(), random_data, logger);
119123
}
120124
NioPeerHandler nio_peer_handler = null;
121-
try { nio_peer_handler = new NioPeerHandler(this.peer_manager); } catch (IOException e) { assert false; }
125+
try {
126+
nio_peer_handler = new NioPeerHandler(this.peer_manager);
127+
} catch (IOException e) {
128+
throw new IllegalStateException("We should never fail to construct nio objects unless we're on a platform that cannot run LDK.");
129+
}
122130
this.nio_peer_handler = nio_peer_handler;
123131
}
124132

0 commit comments

Comments
 (0)