Croway opened a new pull request, #21423: URL: https://github.com/apache/camel/pull/21423
* `.add(new RequestExpectContinue())` was removed to improve test speed execution - most probably it was a copy paste from the apache http 4 documentation. But for our use case is not needed. For reference: ``` The RequestExpectContinue interceptor adds the Expect: 100-continue header to every outgoing HTTP request that has a body. This triggers the following exchange: 1. Client sends request headers (including Expect: 100-continue) and stops — it does not send the body yet 2. Client waits for the server to reply with HTTP/1.1 100 Continue 3. Server is supposed to read the headers, decide if it will accept the body, and send back 100 Continue 4. Client receives 100 Continue and sends the body The problem: the AS2 server side (HttpService in AS2ServerConnection) was not sending back the 100 Continue response in time. So the client waited for the default timeout in HttpRequestExecutor: public static final Timeout DEFAULT_WAIT_FOR_CONTINUE = Timeout.ofSeconds(3); After 3 seconds of silence, the client gave up waiting and sent the body anyway. This happened on every AS2 message exchange, adding exactly 3 seconds per test. For async MDN tests, it happened twice (once for the message, once for the MDN), hence the ~6-second overhead. ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
