Skip to content

Commit

Permalink
chore(deps): update some dependencies (#4019)
Browse files Browse the repository at this point in the history
* chore(deps): update some dependencies

* adjust logging to catch potential ipc issues

* style(fmt): remove outdated comment
  • Loading branch information
imsnif authored Feb 24, 2025
1 parent bdfcfb0 commit 20011a8
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 19 deletions.
28 changes: 14 additions & 14 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 0 additions & 2 deletions zellij-server/src/os_input_output.rs
Original file line number Diff line number Diff line change
Expand Up @@ -387,8 +387,6 @@ impl ClientSender {
for msg in client_buffer_receiver.iter() {
sender.send(msg).with_context(err_context).non_fatal();
}
// If we're here, the message buffer is broken for some reason
log::error!("Client buffer overflow!");
let _ = sender.send(ServerToClientMsg::Exit(ExitReason::Disconnect));
});
ClientSender {
Expand Down
6 changes: 3 additions & 3 deletions zellij-utils/src/ipc.rs
Original file line number Diff line number Diff line change
Expand Up @@ -187,9 +187,9 @@ impl<T: Serialize> IpcSenderWithContext<T> {
if rmp_serde::encode::write(&mut self.sender, &(msg, err_ctx)).is_err() {
Err(anyhow!("failed to send message to client"))
} else {
// TODO: unwrapping here can cause issues when the server disconnects which we don't mind
// do we need to handle errors here in other cases?
let _ = self.sender.flush();
if let Err(e) = self.sender.flush() {
log::error!("Failed to flush ipc sender: {}", e);
}
Ok(())
}
}
Expand Down

0 comments on commit 20011a8

Please sign in to comment.