fix the HTTP test (#892)

Author: TingDaoK

.github/workflows/ci.yml

@@ -199,7 +199,7 @@ jobs:
         ./linux-container-ci.sh ${{ env.BUILDER_VERSION }} aws-crt-${{ matrix.image }} build -p ${{ env.PACKAGE_NAME }}
 
   windows:
-    runs-on: windows-2022 # latest
+    runs-on: windows-2025 # latest
     steps:
     - uses: aws-actions/configure-aws-credentials@v4
       with:
@@ -214,8 +214,8 @@ jobs:
         python -c "from urllib.request import urlretrieve; urlretrieve('${{ env.BUILDER_HOST }}/${{ env.BUILDER_SOURCE }}/${{ env.BUILDER_VERSION }}/builder.pyz?run=${{ env.RUN }}', 'builder.pyz')"
         python builder.pyz build -p ${{ env.PACKAGE_NAME }} --spec=downstream
 
-  windows-vc14:
-    runs-on: windows-2019 # windows-2019 is last env with Visual Studio 2015 (v14.0)
+  windows-vc17:
+    runs-on: windows-2025 # latest
     strategy:
       matrix:
         arch: [x86, x64]
@@ -230,7 +230,7 @@ jobs:
         submodules: true
     - name: Build ${{ env.PACKAGE_NAME }} + consumers
       env:
-        AWS_CMAKE_TOOLSET: v140
+        AWS_CMAKE_TOOLSET: v143 #Visual Studio C++ v143 is Visual Studio 2022 and msvc 17
       run: |
         python -c "from urllib.request import urlretrieve; urlretrieve('${{ env.BUILDER_HOST }}/${{ env.BUILDER_SOURCE }}/${{ env.BUILDER_VERSION }}/builder.pyz?run=${{ env.RUN }}', 'builder.pyz')"
         python builder.pyz build -p ${{ env.PACKAGE_NAME }} downstream
@@ -409,7 +409,7 @@ jobs:
         ./builder localhost-test -p ${{ env.PACKAGE_NAME }} --spec=downstream
 
   localhost-test-win:
-    runs-on: windows-2022 # latest
+    runs-on: windows-2025 # latest
     steps:
     - uses: aws-actions/configure-aws-credentials@v4
       with:

src/test/java/software/amazon/awssdk/crt/test/Http2RequestResponseTest.java

@@ -92,20 +92,22 @@ public TestHttpResponse testHttp2Request(String method, String endpoint, String
 
         Assert.assertNotEquals(-1, response.blockType);
 
-        boolean hasContentLengthHeader = false;
 
         /* The first header of response has to be ":status" for HTTP/2 response */
         response.headers.get(0).getName().equals(":status");
-        for (HttpHeader h : response.headers) {
-            if (h.getName().equals("content-length")) {
-                hasContentLengthHeader = true;
-            }
-        }
 
-        Assert.assertTrue(hasContentLengthHeader);
         if (response.statusCode < 500) { // if the server errored, not our fault
             Assert.assertEquals("Expected and Actual Status Codes don't match", expectedStatus, response.statusCode);
         }
+        if (response.statusCode == 200) {
+            boolean hasContentLengthHeader = false;
+            for (HttpHeader h : response.headers) {
+                if (h.getName().equals("content-length")) {
+                    hasContentLengthHeader = true;
+                }
+            }
+            Assert.assertTrue("Expected to have content-length header that is missing.", hasContentLengthHeader);
+        }
 
         return response;
     }

src/test/java/software/amazon/awssdk/crt/test/HttpRequestResponseTest.java

@@ -90,18 +90,19 @@ public boolean resetPosition() {
 
         Assert.assertNotEquals(-1, response.blockType);
 
-        boolean hasContentLengthHeader = false;
 
-        for (HttpHeader h : response.headers) {
-            if (h.getName().equals("Content-Length")) {
-                hasContentLengthHeader = true;
-            }
-        }
-
-        Assert.assertTrue(hasContentLengthHeader);
         if (response.statusCode < 500) { // if the server errored, not our fault
             Assert.assertEquals("Expected and Actual Status Codes don't match", expectedStatus, response.statusCode);
         }
+        if (response.statusCode == 200) {
+            boolean hasContentLengthHeader = false;
+            for (HttpHeader h : response.headers) {
+                if (h.getName().toLowerCase().equals("content-length")) {
+                    hasContentLengthHeader = true;
+                }
+            }
+            Assert.assertTrue("Expected to have content-length header that is missing.", hasContentLengthHeader);
+        }
 
         return response;
     }
@@ -211,18 +212,29 @@ private void testHttpUpload(boolean chunked) throws Exception {
          *
          */
 
+        // The response is a JSON object, extract the "data" field using proper JSON parsing
         String echoedBody = null;
-        for (String line : body.split("\n")) {
-            String[] keyAndValue = line.split(":", 2);
 
-            // Found JSON Key/Value Pair
-            if (keyAndValue.length == 2) {
-                String key = extractValueFromJson(keyAndValue[0]);
-                String val = extractValueFromJson(keyAndValue[1]);
+        // Find the "data" field in the JSON response
+        int dataIndex = body.indexOf("\"data\"");
+        if (dataIndex != -1) {
+            // Find the colon after "data"
+            int colonIndex = body.indexOf(':', dataIndex);
+            if (colonIndex != -1) {
+                // Find the value after the colon, which should be a quoted string
+                int valueStartIndex = body.indexOf('"', colonIndex);
+                if (valueStartIndex != -1) {
+                    // Find the end of the quoted string
+                    int valueEndIndex = body.indexOf('"', valueStartIndex + 1);
+                    while (valueEndIndex > 0 && body.charAt(valueEndIndex - 1) == '\\') {
+                        // This quote is escaped, find the next one
+                        valueEndIndex = body.indexOf('"', valueEndIndex + 1);
+                    }
 
-                // Found Echoed Body
-                if (key.equals("data")) {
-                    echoedBody = extractValueFromJson(val);
+                    if (valueEndIndex != -1) {
+                        // Extract the value between the quotes
+                        echoedBody = body.substring(valueStartIndex + 1, valueEndIndex);
+                    }
                 }
             }
         }