fix: do not limit grpc response size

crates/lib/src/client.rs

@@ -27,8 +27,8 @@ pub use qcs_api_client_common::configuration::LoadError;
 pub use qcs_api_client_grpc::tonic::Error as GrpcError;
 pub use qcs_api_client_openapi::apis::Error as OpenApiError;
 
-const DEFAULT_MAX_MESSAGE_ENCODING_SIZE: usize = 50 * 1024 * 1024;
-const DEFAULT_MAX_MESSAGE_DECODING_SIZE: usize = 50 * 1024 * 1024;
+/// The maximum size of a gRPC request to the translation service, in bytes.
+const MAX_TRANSLATION_OUTBOUND_REQUEST_SIZE: usize = 50 * 1024 * 1024;
 
 /// A type alias for the underlying gRPC connection used by all gRPC clients within this library.
 /// It is public so that users can create gRPC clients with different APIs using a "raw" connection
@@ -137,8 +137,10 @@ impl Qcs {
         #[cfg(feature = "grpc-web")]
         let channel = wrap_channel_with_grpc_web(service);
         Ok(TranslationClient::new(channel)
-            .max_encoding_message_size(DEFAULT_MAX_MESSAGE_ENCODING_SIZE)
-            .max_decoding_message_size(DEFAULT_MAX_MESSAGE_DECODING_SIZE))
+            .max_encoding_message_size(MAX_TRANSLATION_OUTBOUND_REQUEST_SIZE)
+            // do not limit the received response size, although practically the limit is 4Gb due
+            // to the frame_length of the message being a u32.
+            .max_decoding_message_size(u32::MAX as usize))
     }
 }
 

crates/lib/src/qpu/api.rs

@@ -44,8 +44,8 @@
 
 use crate::client::{GrpcClientError, GrpcConnection, Qcs};
 
-/// The maximum size of a gRPC response, in bytes.
-const MAX_DECODING_MESSAGE_SIZE_BYTES: usize = 250 * 1024 * 1024;
+/// The maximum size of a gRPC request to the controller service, in bytes.
+const MAX_CONTROLLER_OUTBOUND_REQUEST_SIZE: usize = 250 * 1024 * 1024;
 
 pub(crate) fn params_into_job_execution_configuration(
     params: &Parameters,
@@ -367,7 +367,7 @@
 /// to build a custom set of options.
 #[derive(Builder, Clone, Debug, Default, PartialEq)]
 #[allow(clippy::module_name_repetitions)]
 pub struct ApiExecutionOptions {
     /// the inner proto representation
     inner: InnerApiExecutionOptions,
 }
@@ -539,7 +539,10 @@
             .get_qpu_grpc_connection(client, quantum_processor_id)
             .await?;
         Ok(ControllerClient::new(service)
-            .max_decoding_message_size(MAX_DECODING_MESSAGE_SIZE_BYTES))
+            .max_encoding_message_size(MAX_CONTROLLER_OUTBOUND_REQUEST_SIZE)
+            // do not limit the received response size, although practically the limit is 4Gb due
+            // to the frame_length of the message being a u32.
+            .max_decoding_message_size(u32::MAX as usize))
     }
 
     /// Get a GRPC connection to a QPU, without specifying the API to use.

crates/lib/tests/mocked_qpu.rs

@@ -208,8 +208,8 @@ mod mock_qcs {
                 warp::reply::json(&rsp)
             });
 
-        let quantum_processors = warp::path("quantumProcessors")
-            .and(isa.or(default_endpoint).or(accessors));
+        let quantum_processors =
+            warp::path("quantumProcessors").and(isa.or(default_endpoint).or(accessors));
 
         warp::serve(warp::path("v1").and(quantum_processors))
             .run(([127, 0, 0, 1], 8000))