Skip to content

Commit 146a6f5

Browse files
spacekookiemrinalwadhwa
authored andcommitted
fix(rust): context send_message no longer silently eats errors
1 parent 32fa0dd commit 146a6f5

1 file changed

Lines changed: 2 additions & 5 deletions

File tree

  • implementations/rust/ockam/ockam_node/src

implementations/rust/ockam/ockam_node/src/context.rs

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,7 @@ impl Context {
8282
let (reply_tx, mut reply_rx) = channel(1);
8383
let req = NodeMessage::SenderReq(address, reply_tx);
8484

85-
// FIXME/ DESIGN: error communication concept
86-
let _result: Result<()> = match self.sender.send(req).await {
85+
match self.sender.send(req).await {
8786
Ok(()) => {
8887
if let Some(NodeReply::Sender(_, s)) = reply_rx.recv().await {
8988
let msg = msg.encode().unwrap();
@@ -96,9 +95,7 @@ impl Context {
9695
}
9796
}
9897
Err(_e) => Err(Error::FailedSendMessage.into()),
99-
};
100-
101-
Ok(())
98+
}
10299
}
103100

104101
/// Block the current worker to wait for a typed message

0 commit comments

Comments
 (0)