aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorIvan Molodetskikh <yalterz@gmail.com>2024-06-20 09:22:02 +0300
committerIvan Molodetskikh <yalterz@gmail.com>2024-09-01 23:47:19 -0700
commit397b7e4bb942b7fe3bd6f1652fa31ccec3ecc48e (patch)
treee8d8ef82522f21922d3c18fa1b09be9417dda226
parent598b27f83c4d4dd72c5cfbd06ca60b12e2833429 (diff)
downloadniri-397b7e4bb942b7fe3bd6f1652fa31ccec3ecc48e.tar.gz
niri-397b7e4bb942b7fe3bd6f1652fa31ccec3ecc48e.tar.bz2
niri-397b7e4bb942b7fe3bd6f1652fa31ccec3ecc48e.zip
ipc: Read only a single line on the client
Allow extensibility.
-rw-r--r--niri-ipc/src/socket.rs12
-rw-r--r--src/ipc/server.rs3
2 files changed, 9 insertions, 6 deletions
diff --git a/niri-ipc/src/socket.rs b/niri-ipc/src/socket.rs
index 67b9625c..3964f000 100644
--- a/niri-ipc/src/socket.rs
+++ b/niri-ipc/src/socket.rs
@@ -1,7 +1,7 @@
//! Helper for blocking communication over the niri socket.
use std::env;
-use std::io::{self, Read, Write};
+use std::io::{self, BufRead, BufReader, Write};
use std::net::Shutdown;
use std::os::unix::net::UnixStream;
use std::path::Path;
@@ -50,14 +50,16 @@ impl Socket {
pub fn send(self, request: Request) -> io::Result<Reply> {
let Self { mut stream } = self;
- let mut buf = serde_json::to_vec(&request).unwrap();
- stream.write_all(&buf)?;
+ let mut buf = serde_json::to_string(&request).unwrap();
+ stream.write_all(buf.as_bytes())?;
stream.shutdown(Shutdown::Write)?;
+ let mut reader = BufReader::new(stream);
+
buf.clear();
- stream.read_to_end(&mut buf)?;
+ reader.read_line(&mut buf)?;
- let reply = serde_json::from_slice(&buf)?;
+ let reply = serde_json::from_str(&buf)?;
Ok(reply)
}
}
diff --git a/src/ipc/server.rs b/src/ipc/server.rs
index 5b614a83..5d2c8524 100644
--- a/src/ipc/server.rs
+++ b/src/ipc/server.rs
@@ -132,7 +132,8 @@ async fn handle_client(ctx: ClientCtx, stream: Async<'_, UnixStream>) -> anyhow:
}
}
- let buf = serde_json::to_vec(&reply).context("error formatting reply")?;
+ let mut buf = serde_json::to_vec(&reply).context("error formatting reply")?;
+ buf.push(b'\n');
write.write_all(&buf).await.context("error writing reply")?;
Ok(())