apollo_infra: add request max size tests

Author: Itay-Tsabary-Starkware

crates/apollo_infra/src/component_definitions.rs

@@ -119,6 +119,8 @@ where
 pub enum ServerError {
     #[error("Could not deserialize client request: {0}")]
     RequestDeserializationFailure(String),
+    #[error("Request body too large: {0}")]
+    RequestBodyTooLarge(String),
 }
 
 #[derive(Debug)]

crates/apollo_infra/src/component_server/remote_component_server.rs

@@ -190,10 +190,8 @@ where
             match Limited::new(http_request.into_body(), max_request_body_bytes).collect().await {
                 Ok(collected) => collected.to_bytes(),
                 Err(err) => {
-                    error!("Failed to collect request body: {err}");
-                    let server_error = ServerError::RequestDeserializationFailure(
-                        "Request body too large".to_string(),
-                    );
+                    warn!("Request body too large: {err}");
+                    let server_error = ServerError::RequestBodyTooLarge(err.to_string());
                     return Ok(HyperResponse::builder()
                         .status(StatusCode::PAYLOAD_TOO_LARGE)
                         .header(CONTENT_TYPE, APPLICATION_OCTET_STREAM)

crates/apollo_infra/src/tests/remote_component_client_server_test.rs

@@ -784,3 +784,107 @@ async fn tcp_keepalive_idle_time_matches_config() {
         "TCP_KEEPIDLE should equal keepalive_timeout_ms * TCP_IDLE_TIMEOUT_FACTOR"
     );
 }
+
+/// Server rejects a request whose body exceeds `max_request_body_bytes` with 413 and
+/// `ServerError::RequestBodyTooLarge`.
+#[tokio::test]
+async fn request_body_too_large() {
+    let mut available_ports = available_ports_factory(unique_u16!());
+    let a_socket = available_ports.get_next_local_host_socket();
+    let dummy_b_socket = available_ports.get_next_local_host_socket();
+
+    // B client points at a non-existent server; it will never be called because the oversized
+    // request is rejected at the HTTP layer before any component logic runs.
+    let b_remote_client = ComponentBClient::new(
+        RemoteClientConfig::default(),
+        &dummy_b_socket.ip().to_string(),
+        dummy_b_socket.port(),
+        &TEST_REMOTE_CLIENT_METRICS,
+    );
+    let component_a = ComponentA::new(Box::new(b_remote_client));
+
+    let (tx_a, rx_a) = channel::<RequestWrapper<ComponentARequest, ComponentAResponse>>(32);
+    let a_local_client = LocalComponentClient::new(tx_a, &TEST_LOCAL_CLIENT_METRICS);
+
+    let mut local_server = LocalComponentServer::new(
+        component_a,
+        &LocalServerConfig::default(),
+        rx_a,
+        &TEST_LOCAL_SERVER_METRICS,
+    );
+    task::spawn(async move {
+        let _ = local_server.start().await;
+    });
+
+    let server_config = RemoteServerConfig { max_request_body_bytes: 1, ..Default::default() };
+    let mut remote_server = RemoteComponentServer::new(
+        a_local_client,
+        server_config,
+        a_socket.port(),
+        &TEST_REMOTE_SERVER_METRICS,
+    );
+    task::spawn(async move {
+        let _ = remote_server.start().await;
+    });
+    task::yield_now().await;
+
+    let uri: Uri = format!("http://[{}]:{}/", a_socket.ip(), a_socket.port()).parse().unwrap();
+    let http_request = Request::post(uri)
+        .header(CONTENT_TYPE, APPLICATION_OCTET_STREAM)
+        .header(REQUEST_ID_HEADER, RequestId::generate().to_string())
+        .body(Full::new(Bytes::from("x".repeat(1024))))
+        .unwrap();
+    let http_response =
+        Client::builder(TokioExecutor::new()).build_http().request(http_request).await.unwrap();
+
+    assert_eq!(http_response.status(), StatusCode::PAYLOAD_TOO_LARGE);
+    let body_bytes = http_response.into_body().collect().await.unwrap().to_bytes();
+    let server_error = SerdeWrapper::<ServerError>::wrapper_deserialize(&body_bytes).unwrap();
+    assert!(matches!(server_error, ServerError::RequestBodyTooLarge(_)));
+}
+
+/// Client returns `ResponseParsingFailure` when the server's response body exceeds
+/// `max_response_body_bytes`.
+#[tokio::test]
+async fn response_body_too_large() {
+    let socket = available_ports_factory(unique_u16!()).get_next_local_host_socket();
+    task::spawn(async move {
+        async fn handler(
+            _http_request: Request<Incoming>,
+        ) -> Result<Response<Full<Bytes>>, Infallible> {
+            Ok(Response::builder()
+                .status(StatusCode::OK)
+                .header(CONTENT_TYPE, APPLICATION_OCTET_STREAM)
+                .body(Full::new(Bytes::from(vec![0u8; 1024])))
+                .unwrap())
+        }
+
+        let listener = TcpListener::bind(&socket).await.unwrap();
+        loop {
+            let Ok((stream, _)) = listener.accept().await else { continue };
+            let io = TokioIo::new(stream);
+            let service = service_fn(|req| async move { handler(req).await });
+            tokio::spawn(async move {
+                let _ = Http2ServerBuilder::new(TokioExecutor::new())
+                    .http2()
+                    .serve_connection(io, service)
+                    .await;
+            });
+        }
+    });
+    task::yield_now().await;
+
+    let client_config =
+        RemoteClientConfig { max_response_body_bytes: 1, retries: 0, ..FAST_FAILING_CLIENT_CONFIG };
+    let client = ComponentAClient::new(
+        client_config,
+        &socket.ip().to_string(),
+        socket.port(),
+        &TEST_REMOTE_CLIENT_METRICS,
+    );
+
+    let Err(error) = client.a_get_value().await else {
+        panic!("Expected an error");
+    };
+    assert!(matches!(error, ClientError::ResponseParsingFailure(_)), "unexpected error: {error}");
+}