MonitoringResponseOutOfOrderStrategy: Always perform a blocking read to check for data

Author: rschmitt

httpcore5-testing/src/test/java/org/apache/hc/core5/testing/classic/MonitoringResponseOutOfOrderStrategyIntegrationTest.java

@@ -78,6 +78,7 @@ public MonitoringResponseOutOfOrderStrategyIntegrationTest(final URIScheme schem
                 .setSocketConfig(SocketConfig.custom()
                         .setSoTimeout(TIMEOUT)
                         .setSoKeepAlive(false)
+                        .setTcpNoDelay(true)
                         .build())
                 .setRequestRouter(RequestRouter.<HttpRequestHandler>builder()
                         .addRoute(RequestRouter.LOCAL_AUTHORITY, "*", (request, response, context) -> {

httpcore5/src/main/java/org/apache/hc/core5/http/impl/io/MonitoringResponseOutOfOrderStrategy.java

@@ -92,8 +92,7 @@ public boolean isEarlyResponseDetected(
             final long totalBytesSent,
             final long nextWriteSize) throws IOException {
         if (nextWriteStartsNewChunk(totalBytesSent, nextWriteSize)) {
-            final boolean ssl = connection.getSSLSession() != null;
-            return ssl ? connection.isDataAvailable(Timeout.ONE_MILLISECOND) : (inputStream.available() > 0);
+            return connection.isDataAvailable(Timeout.ONE_MILLISECOND);
         }
         return false;
     }