From 7e3fbec077cb813a15c1ec3308488a5cfafe13e7 Mon Sep 17 00:00:00 2001 From: Michael Zimmermann Date: Wed, 20 Jan 2021 18:00:12 +0100 Subject: [PATCH] example-service: set max frame length to usize::MAX I don't know what the intention was behind using u32::MAX + 1 but since the argument's type is usize this is the only giant value that makes sense to me. --- example-service/src/client.rs | 2 +- example-service/src/server.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/example-service/src/client.rs b/example-service/src/client.rs index cd08a18..0ff853d 100644 --- a/example-service/src/client.rs +++ b/example-service/src/client.rs @@ -43,7 +43,7 @@ async fn main() -> io::Result<()> { let name = flags.value_of("name").unwrap().into(); let mut transport = tarpc::serde_transport::tcp::connect(server_addr, Json::default); - transport.config_mut().max_frame_length(4294967296); + transport.config_mut().max_frame_length(usize::MAX); // WorldClient is generated by the service attribute. It has a constructor `new` that takes a // config and any Transport as input. diff --git a/example-service/src/server.rs b/example-service/src/server.rs index fd12d60..59f9ed0 100644 --- a/example-service/src/server.rs +++ b/example-service/src/server.rs @@ -58,7 +58,7 @@ async fn main() -> io::Result<()> { // JSON transport is provided by the json_transport tarpc module. It makes it easy // to start up a serde-powered json serialization strategy over TCP. let mut listener = tarpc::serde_transport::tcp::listen(&server_addr, Json::default).await?; - listener.config_mut().max_frame_length(4294967296); + listener.config_mut().max_frame_length(usize::MAX); listener // Ignore accept errors. .filter_map(|r| future::ready(r.ok()))